Skip to content

fix: fullscreen overlay issues with header, map, and controls (#887, #859, #829)#893

Closed
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:fix/fullscreen-overlay-bugs
Closed

fix: fullscreen overlay issues with header, map, and controls (#887, #859, #829)#893
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:fix/fullscreen-overlay-bugs

Conversation

@NewCoder3294
Copy link
Collaborator

Summary

Test Plan

  • Enter panel fullscreen (e.g., Live News or Webcams) → variant switcher in header is clickable
  • Enter webcam fullscreen → no map content visible behind it
  • Enter any panel fullscreen → map legend and layer toggles are not visible
  • Exit fullscreen → everything returns to normal

🤖 Generated with Claude Code

…map controls (koala73#887, koala73#859, koala73#829)

- Give header z-index above fullscreen panels so variant switcher stays clickable (koala73#887)
- Hide map section when a panel is fullscreen (koala73#859)
- Ensure map legend and layer toggles are hidden during fullscreen (koala73#829)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 3, 2026

@NewCoder3294 is attempting to deploy a commit to the Elie Team on Vercel.

A member of the Team first needs to authorize it.

@koala73 koala73 added High Value Meaningful contribution to the project Ready to Merge PR is mergeable, passes checks, and adds value labels Mar 3, 2026
@koala73
Copy link
Owner

koala73 commented Mar 19, 2026

Superseded by #1313 which takes a better approach: hiding the entire .map-section container with a single rule instead of individual elements. That PR has been rebased onto main and is ready for review.

@koala73 koala73 closed this Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Value Meaningful contribution to the project Ready to Merge PR is mergeable, passes checks, and adds value

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants