0.73: Allow fractional zoom stopping with Map.TouchZoom and fix CRS.scale calc with fractional zoom #3325
Conversation
This file contains 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
…cale calculation with fractional zoom level
adriankb
changed the title
|
This will break a lot of apps, especially ones that use tiles (which is 99% of Leaflet apps). So I don't want to merge it into stable — lets continue discussion on the master changes. |
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.
Hi.
I believe that I have found a fix to allow touch/pinch fractional zoom support (when using plain Leaflet object and or GeoJson objects). But I could not comment as to if this will break any map with tiles. We currently don't use tiles, so I don't have expertise.
This Pull is for stable (sorry if this is the wrong destination), I will create a second for master #3326 (as the code is different). I believe work has been done in master for version 1.0 to support better zooming so if this Pull is ignored in favour of the master pull then at least this code might serve as a reference for others using 0.7.
The underlying issue being solved is the "jump" in the map when stopping a touch/pinch zoom whilst the map snaps to a whole number zoom level. Our users on iPad's and Android were finding this very obvious and it was very hard for them to zoom in/out to the level of detail that they wanted.
Changes to Map.TouchZoom support stopping the zoom level exactly where the user has stopped the pinch.
The secondary issue uncovered is that the built in CRS.scale calculation also does not support fractional zoom. The side-affect of this is that when touch/pinch zooming, the percent with which the map is grown or shrunk does not reflect the final size that the map will be drawn at. Thus this means that there is still a "jump" in the map when stopping a touch/pinch zoom, even with the first fix in place.
The commit to CRS.js supports Scale calculation for fractional zoom levels.
Please refer to these related issues:
kartena/Proj4Leaflet#57
kartena/Proj4Leaflet#73
#426
#1309
http://stackoverflow.com/questions/24151177/fractional-zoom-levels
#2382
#2558