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

During setup, sort real-time weather stations by adding the three closest ones first #51

Open
Rudd-O opened this issue Aug 23, 2024 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@Rudd-O
Copy link
Owner

Rudd-O commented Aug 23, 2024

During setup, we show a list of No station, then the default selected is the closest station, then we sort alphabetically.

This doesn't make much sense, since we should be offering the user the three closest stations (with the closest station preselected) when we show the form to the user, and then the rest in alphabetical order (with No station being first, always).

We do offer the choice of station this because sometimes the closest station is like 2 KM above the elevation of the user, which would give the user terrible, inaccurate data for his location. Accordingly, to facilitate this for the user, we should show the three closest stations right under his mouse when he clicks the dropdown.

Perhaps we should also show the distance of the station to the user in the same screen, to aid in selection.

This may require changes in hamsclientfork to provide for topological distance sorting of the entities as returned by its get all stations call.

@Rudd-O Rudd-O added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant