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

Zoom levels too low #47

Open
plepe opened this issue Aug 2, 2022 · 3 comments
Open

Zoom levels too low #47

plepe opened this issue Aug 2, 2022 · 3 comments
Labels

Comments

@plepe
Copy link
Owner

plepe commented Aug 2, 2022

Thanks!
BTW I noticed the minimum zoom level to be way too low to remain useful if you apply (such) filters. That's probably not only applicable to bicycle amenities but pretty much everything where you have set minimum zoom levels.

Originally posted by @stefanct in #46 (comment)

@plepe
Copy link
Owner Author

plepe commented Aug 2, 2022

@stefanct Yes, you are right. Especially on a smartphone with small display. When setting the zoom levels, I had a desktop browser in mind.

There are two alternatives to solve this problem:

  • Set higher zoom levels in the categories
  • Add a setting, where you can override the current zoom level setting (force z12, although z17 is active)

Maybe both?

@plepe plepe added the bug label Aug 2, 2022
@stefanct
Copy link

stefanct commented Aug 2, 2022

I noticed on a laptop (w/o very high DPI)... I don't exactly see the connection between the device and the zoom level TBH, so maybe there was some miscommunication. What I meant is that the queries don't get executed when you zoom out too much. If there are many POIs this makes sense but as soon as you have only a few per city and want to get an overview of larger regions you are stuck and have to either pan endlessly or use overpass turbo etc. instead of OSB.

Why not just query and only show the results if they are below a certain limit (per pixel area)?

@plepe
Copy link
Owner Author

plepe commented Sep 13, 2022

Sorry, forgot to reply.

The idea is to reduce traffic to the database server. So, I don't know about the density before querying the area ... and still, as I'm only asking for the first 250 (or so) result, I don't know if it is just over 250 or 10.000.

I'll think about improvements.

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

No branches or pull requests

2 participants