Fix #2042 by aligning minimap zoom handling between interface button and hotkey handlers
#2042 by aligning minimap zoom handling between interface button and hotkey handlers
veloren/dev/veloren:feriixu/fix_minimap_zooming into master
Fixes #2042 (closed) by aligning the implementations of the minimap zoom level clamping.
Should this fix appear in the changelog? If so, please let me know and I shall add a short notice.
What I did:
- Align minimap zoom handling for interface buttons and hotkeys.
- Ensure minimap zoom stays within limits using the
clamp
method at both code locations. - Duplicate minimap zoom handling code, from correctly working interface handler, to hotkey handler.
- Improve logging. (Not sure if wanted, please advise)
- Include TODOs to remind developers to consolidate duplicated code in the future.
Edited by F. St.