Map: Fix various zoomLevel problems #12870
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
|
I think 15x might not be quite enough, at least in my case we move around within a radius of like 15m. My original plan was to limit zoom to the QGeoCameraCapabilities.maximumZoomLevel |
Collaborator
Author
|
The 15 is just when loading a plan file. You can still manually zoom in all the way to 20. Problem is not all providers go up that far. |
Collaborator
Author
|
I'm going to look at plumbing the max zoom level up through the providers so that it's real/variable as opposed to hardcoded. |
Collaborator
Author
|
Turns out tiles vary by location as far as zoom level availability. So the problem I was seeing is location specific. I'll change this to no go past the global max of 20. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Discovered while testing #12236