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

Leaflet Map cannot be zoomed after scale 1:2257 #1639

Closed
tdipisa opened this issue Mar 27, 2017 · 5 comments
Closed

Leaflet Map cannot be zoomed after scale 1:2257 #1639

tdipisa opened this issue Mar 27, 2017 · 5 comments

Comments

@tdipisa
Copy link
Member

tdipisa commented Mar 27, 2017

The bug can be replicated using the scalebar.
This works fine using openlayers.

@MV88
Copy link
Contributor

MV88 commented Mar 27, 2017

I wanted to customize the scale list but I discovered some problems.
The fact is that the scales are defined in the connect method of the scalebox plugin.
Thus they are not overridable from the config.

This is a general problem and should be opened a proper issue in order to fix this.
see #1645

@MV88
Copy link
Contributor

MV88 commented Jul 12, 2017

Overriding the scales is not a good solution, thus this issue need to be investigated more

@offtherailz
Copy link
Member

The problem could be resolved setting maxZoom and maxNativeZoom for each TileLayer configuration and overriding them properly in the map components.
Reference

@MV88
Copy link
Contributor

MV88 commented Aug 31, 2017

This issue is more tricky than expected.
I can set maxZoom and maxNativeZoom with different defaults on Google and OSM with no problems.

some issues raise here:

  • The Tileprovider layer can have different max native zoom (nasa 8, opentopomap around 16). in this case we should save also the maxNativeZoom for each of them. I accept suggestions.
  • We also need to update the scalebar when the background is changed.
  • Actually if we change the background from OSM with zoom level 21 to google layer, we need to change the zoom level to its max (19) if not available.

All of the above should be done only for leaflet though

@tdipisa tdipisa assigned kappu72 and unassigned MV88 Sep 1, 2017
@MV88
Copy link
Contributor

MV88 commented Sep 7, 2017

see #394

kappu72 pushed a commit to kappu72/MapStore2 that referenced this issue Sep 11, 2017
offtherailz pushed a commit that referenced this issue Sep 12, 2017
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

4 participants