Adaptive hill shading: Remove the default maximum zoom limits by Sublimis · Pull Request #1612 · mapsforge/mapsforge
| public class MapViewPosition extends Observable implements Persistable { | ||
|
|
||
| public static final int ZOOM_MIN_DEFAULT = 0; | ||
| public static final int ZOOM_MAX_DEFAULT = 22; |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Sublimis Please do not put such custom zoom limits.
Users can limit the max zoom with the related method.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you leave this commit out, or should we push another one that will revert it?
Is the unlimited maximum zoom in the app samples also intentional or can we limit it there? To prevent zooming indefinitely, while the scale remains at "1 meter". But it doesn't really matter, it's just an inconvenience.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Sublimis Better make a new pull request if cannot revert the commit?