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

refresh data icon on radar graphic #1184

Open
huss opened this issue Apr 3, 2024 · 0 comments
Open

refresh data icon on radar graphic #1184

huss opened this issue Apr 3, 2024 · 0 comments
Labels
i-good-first-issue This issue is probably a good starting point for people new to coding or the OED project. p-medium-priority t-bug The issue is related to an aspect of the software which is incorrect
Milestone

Comments

@huss
Copy link
Member

huss commented Apr 3, 2024

Describe the bug

Only the line, bar and radar graphic pages have the refresh data icon to get new reading data for the specified zoom range. This was originally designed for the line graphic (before RTK). It works reasonably for bar so is fine to keep there. However, when you zoom on radar is changes the y-axis (not the x-axis for dates) so refresh does not make sense.

Expected behavior

The refresh icon should be removed from the radar graphic.

Screenshots

Remove the icon in top, right with red box around it.

Screenshot 2024-04-03 103605

Additional context

None

@huss huss added t-bug The issue is related to an aspect of the software which is incorrect p-medium-priority i-good-first-issue This issue is probably a good starting point for people new to coding or the OED project. labels Apr 3, 2024
@huss huss added this to the 1.1 release milestone Apr 3, 2024
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-medium-priority t-bug The issue is related to an aspect of the software which is incorrect
Projects
Status: Todo
Development

No branches or pull requests

1 participant