Skip to content

Zeppelin 1283 - #1695

Closed
PetroKostyuk wants to merge 3 commits into
apache:masterfrom
PetroKostyuk:ZEPPELIN-1283
Closed

Zeppelin 1283#1695
PetroKostyuk wants to merge 3 commits into
apache:masterfrom
PetroKostyuk:ZEPPELIN-1283

Conversation

@PetroKostyuk

Copy link
Copy Markdown

What is this PR for?

It's adding option to specify column in scatterPlot, that will be displayed in label

What type of PR is it?

Feature

Todos

I had trouble to get label displayed for most of points. Label was triggered by cursor on different position than point was. See picture, my cursor was on position of red X. But this problem was there even before my edits and I think it should be handeled as separate issue. Mostly I was able to trigger labels when I selected only Y axis and hovered on points on bottom of chart.
screenshot from 2016-11-26 13-51-10

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1283

How should this be tested?

reate scatterPlot and put some column in new 'label' field. Values from that column should appear in label corresponding to selected point.

Screenshots (if appropriate)

screenshot from 2016-11-28 19-51-52
screenshot from 2016-11-28 19-51-18

@bzz

bzz commented Dec 1, 2016

Copy link
Copy Markdown
Member

@PetroKostyuk thank you for improvement!

It looks like your work need to be rebased on the latest master in order to resolve conflicts, before merging.

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@PetroKostyuk@bzz