Skip to content

fix(core): fix width and height of map overlay#587

Open
dopenguin wants to merge 3 commits intonextfrom
fix/map-overlay
Open

fix(core): fix width and height of map overlay#587
dopenguin wants to merge 3 commits intonextfrom
fix/map-overlay

Conversation

@dopenguin
Copy link
Copy Markdown
Member

Summary

Update the width and the height of the map overlay. This was broken after #545.
Note that the overlay now is placed below the interface elements. I think this is quite nice. If you disagree, I'll change it back to the previous implementation.

Instructions for local reproduction and review

npm run snowbox and try scrolling via mousewheel.

@dopenguin dopenguin added this to the POLAR@3 milestone Mar 9, 2026
@dopenguin dopenguin self-assigned this Mar 9, 2026
@dopenguin dopenguin added the bug Something isn't working label Mar 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 9, 2026

PR Preview Action v1.8.0

QR code for preview link

🚀 View preview at
https://Dataport.github.io/polar/pr-preview/pr-587/

Built to branch gh-pages at 2026-03-30 10:56 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@dopenguin dopenguin force-pushed the fix/map-overlay branch 2 times, most recently from f34ac73 to e334405 Compare March 17, 2026 16:12
Copy link
Copy Markdown
Member

@warm-coolguy warm-coolguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏓 @dopenguin

Due to "Press Ctrl + Scroll to zoo", I prefer the prior solution of overlapping it all.

Image displaying that the current client cuts off the scroll instructions if UI elements are above it, rendering it ineffective

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants