Skip to content

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
F. St.requested to merge
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.

Merge request reports

Loading