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

Radiussearch #134

Open
woheller69 opened this issue Sep 17, 2020 · 2 comments
Open

Radiussearch #134

woheller69 opened this issue Sep 17, 2020 · 2 comments

Comments

@woheller69
Copy link
Contributor

What about a radius search result like this?
It is based on the cities layer of leaflet-openweathermap.js (https://github.com/buche/leaflet-openweathermap)
I have a prototype in my branch radiussearch.

radiussearch

@sleep-yearning
Copy link
Member

That is a much more visualizing variant.
I think if it contains all the data of the previous display this would be better.
One thing to consider is readability though, the old display was very easy to read and this map view should also be affected by text size and such.

@woheller69
Copy link
Contributor Author

I tends to consume a lot of API calls. In my RadarWeather https://github.com/woheller69/weather I have implemented a different approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants