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

Remove most plotly icons from radar & map graphic #1161

Open
huss opened this issue Mar 21, 2024 · 1 comment
Open

Remove most plotly icons from radar & map graphic #1161

huss opened this issue Mar 21, 2024 · 1 comment
Assignees
Labels
i-good-first-issue This issue is probably a good starting point for people new to coding or the OED project. p-low-priority t-enhancement This issues tracks a potential improvement to the software
Milestone

Comments

@huss
Copy link
Member

huss commented Mar 21, 2024

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

PR #1113 removed most Plotly icons. The radar graphic still has three of the icons. On map they appear as the mouse is moved over the graphic. They should be made consistent with the other graphics.

Describe the solution you'd like

See above.

Describe alternatives you've considered

On Radar, one icon appears to do nothing and one lists the Plotly version so those don't hurt much. The camera is actually desired. On map, it is as before with all the icons. Thus, OED can leave as is until there is a fix.

Additional context

Issue #1160 proposes to add the graphic capture icon to all graphics and this should be consistent with that one.

@huss huss added t-enhancement This issues tracks a potential improvement to the software p-low-priority i-good-first-issue This issue is probably a good starting point for people new to coding or the OED project. labels Mar 21, 2024
@huss huss changed the title Remove most plotly icons from radar graphic Remove most plotly icons from radar & map graphic Mar 21, 2024
@huss huss moved this to Todo in Redux Toolkit Mar 23, 2024
@huss huss added this to the 1.1 release milestone Mar 23, 2024
@williamw04
Copy link
Contributor

I'll attempt this problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i-good-first-issue This issue is probably a good starting point for people new to coding or the OED project. p-low-priority t-enhancement This issues tracks a potential improvement to the software
Projects
Status: Todo
Development

No branches or pull requests

2 participants