-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Implement legend.grouptitlefont
and hoverlabel.grouptitlefont
#6040
Conversation
Behavior looks great. My only question is about the attribute names/locations:
|
Actually I did try the exact same changes but ran into some problems.
|
Probably just needs excluding here: plotly.js/src/plot_api/plot_api.js Lines 1722 to 1723 in 10d3930
Ah right, because we supply legend defaults after traces, which we need to do so that we know whether to show the legend or not. I still think this is worthwhile, how about this: coerce |
legendgrouptitlesfont
and hoverlabel.grouptitlesfont
legend.grouptitlesfont
and hoverlabel.grouptitlesfont
legend.grouptitlesfont
and hoverlabel.grouptitlesfont
legend.grouptitlefont
and hoverlabel.grouptitlefont
Now we have the implentation for |
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.
💃 Thanks for the tweaks :)
Resolves #5920.
@plotly/plotly_js