Skip to content

canvas: fix minimap hiding#4098

Merged
tvanderstad merged 1 commit intomasterfrom
canvas-input
Jan 14, 2026
Merged

canvas: fix minimap hiding#4098
tvanderstad merged 1 commit intomasterfrom
canvas-input

Conversation

@tvanderstad
Copy link
Copy Markdown
Contributor

@tvanderstad tvanderstad commented Jan 14, 2026

Fixes an issue where pan & pinch on iPad do not work on the region where the minimap would be shown when the minimap is hidden. Does this by extracting and using a new fn should_show_mini_map.

QA:

  • pan using the minimap
  • pan canvas in the minimap area when it's off
  • pan canvas in the minimap area when not in notebook mode
  • pan canvas in the minimap area when overlay is hidden

side note: it's sensible to only show the minimap in scroll mode but confusing that you can have show minimap on and not see the minimap

@tvanderstad tvanderstad requested a review from ad-tra January 14, 2026 17:27
@tvanderstad tvanderstad merged commit 01d244b into master Jan 14, 2026
1 check passed
@tvanderstad tvanderstad deleted the canvas-input branch January 14, 2026 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant