-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Enable text for scattergl trace #2737
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 32 commits
8ec98b9
862905c
d1a06c0
cbc7f3f
cc2cca1
e5a771c
349b551
71c1acb
4fe4e3a
ff1d4fc
95a3d1a
ed7e0a0
fa96cc7
9653fcf
17c1b0b
0154571
c47d258
79eae53
bd4491c
741bfb1
4d67fa4
90479a7
6def5e4
47cf7e2
a53baf6
b3fb445
e09a4e1
668d7ee
a41ea35
1702b30
62bf263
65f97f3
55906cc
46bc623
7d75169
1a64005
bdc8f4a
959a555
431c564
9f3827c
3af4d2f
a83077b
29fa2ef
e462eda
52614d1
143a61e
fb08f55
a5637dd
6aedaa6
235f1ac
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -37,9 +37,15 @@ var attrs = module.exports = overrideAll({ | |
'this trace\'s (x,y) coordinates.' | ||
].join(' ') | ||
}), | ||
|
||
hovertext: scatterAttrs.hovertext, | ||
|
||
textposition: scatterAttrs.textposition, | ||
textfont: scatterAttrs.textfont, | ||
|
||
mode: { | ||
valType: 'flaglist', | ||
flags: ['lines', 'markers'], | ||
flags: ['lines', 'markers', 'text'], | ||
extras: ['none'], | ||
role: 'info', | ||
description: [ | ||
|
@@ -77,10 +83,12 @@ var attrs = module.exports = overrideAll({ | |
hoveron: scatterAttrs.hoveron, | ||
|
||
selected: { | ||
marker: scatterAttrs.selected.marker | ||
marker: scatterAttrs.selected.marker, | ||
textfont: scatterAttrs.selected.textfont | ||
}, | ||
unselected: { | ||
marker: scatterAttrs.unselected.marker | ||
marker: scatterAttrs.unselected.marker, | ||
textfont: scatterAttrs.unselected.textfont | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does this work? And what about There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Confirmed. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. mostly done in 1a64005 |
||
}, | ||
|
||
opacity: plotAttrs.opacity | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that scattergl supports on-graph, we can add
hovertext
like in svg scatter traces:plotly.js/src/traces/scatter/attributes.js
Lines 91 to 104 in 6ac47ec
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done in 959a555