-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Update menus open items #810
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
Comments
Yep. Here's someone struggling with this out-in-the-wild. Also came up from a customer @charleyferrari chatted up today. |
Closing #890 in favor of this. I'll only add that if menus scrolled, that would presumably also solve the hard limit of six items (a very hard limit—the menu disappears if you exceed six). Edit: one more thing: I found the menu takes up too much vertical space where it should instead probably just overlap the plot when expanded. There may be a config option to work around this, but unless there's a reason I'm not seeing, it should perhaps be the default. See #890 for screenshot and example. |
@etpinard — What do you think about changing the expanded menu to HTML? It will be a bit awkward to position relative to the top-level SVG container, but it would probably be significantly more accessible and require 80% less code. Plus, it's not clear to me why you'd want the expanded menu to be rendered to SVG. Just a thought as I look through what it means to implement complex controls in SVG. |
Another user who is having trouble accessing menu items past the 10th one or so: https://plot.ly/~wholleran/829/completion-percentage-vs-period/?share_key=J0gquGi2vrEhTurmsYsP3s |
Hey @n-riesco - Can you take this before you go onto animations for bars? (At least making it scroll-able). Thanks! |
Layout menu feature PR: #770
Open items:
method
or just let Plotly[method] error out?direction
see Add button type to updatemenus #974 (comment)The text was updated successfully, but these errors were encountered: