-
-
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
Add legend itemsizing
#3732
Add legend itemsizing
#3732
Conversation
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.
@etpinard thanks for the PR.
Concerning those gl3d baseline changes, I am wondering if we may apply half of the sizes?
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.
I love legend-constant-itemsizing
baseline!
Few suggestions related to waterfall
mock are:
OK. My comments above were non-blocking. |
Awesome. |
|
resolves #3727 - by adding
legend.itemsizing
with values'constant'
and'trace'
(the default and current behavior).I chose
itemsizing
oversizemode
as we already have asizemode
attribute in thescatter*
traces which does something sufficiently different in my mind to not replicate the name over tolayout.legend
. Let me know if you can think of a better name.cc @plotly/plotly_js @nicolaskruchten