Skip to content
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 Display More Plotly Button #1358

Open
williamw04 opened this issue Oct 11, 2024 · 1 comment
Open

Add Display More Plotly Button #1358

williamw04 opened this issue Oct 11, 2024 · 1 comment
Labels
p-low-priority t-enhancement This issues tracks a potential improvement to the software
Milestone

Comments

@williamw04
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Currently the graphics has a bunch of useful plotly icons. However, most users are only going to use the png plotly feature and the other features may seem overwhelming. Using plotly implement a button that would display the advanced buttons and hide them.

Describe the solution you'd like

see above

Describe alternatives you've considered

Currently the icons are being controlled in the config in the plotly object declaration.
The current config settings are:
displayModeBar: true, //displays the plotly buttons
modeBarButtonsToRemove: ['select2d','lasso2d','autoScale2d','resetScale2d'], //remove unnecessary buttons

The plotly objects are created then displayed/drawn and changing the object that was displayed doesn't reflect onto the site. However, we can maybe get around this by using redux and tracking the state so that when there's a change the entire site is redrawn.

Additional context

This feature is related to issue #1161.

@huss huss added t-enhancement This issues tracks a potential improvement to the software p-low-priority labels Oct 11, 2024
@huss huss added this to the 2.x milestone Oct 11, 2024
@huss
Copy link
Member

huss commented Dec 23, 2024

This was mostly solved by PR #1329 except for maps. That can be done once the update to RTK is done and there are comments in the code about doing that. Issue #1161 about the map icons not always bring present should be resolved by this but it should be checked that the icons are always present as on other graphics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p-low-priority t-enhancement This issues tracks a potential improvement to the software
Projects
None yet
Development

No branches or pull requests

2 participants