Skip to content

fix(lightbox): maintain 44px mobile touch targets and hide main under overlay (#901) - #916

Open
ralksta wants to merge 1 commit into
devfrom
ux/901-mobile-lightbox-touch-target-overlay
Open

ralksta wants to merge 1 commit into
devfrom
ux/901-mobile-lightbox-touch-target-overlay

Conversation

@ralksta

@ralksta ralksta commented Nov 10, 2026

Copy link
Copy Markdown
Owner

Fixes #901.

Changes

  • In components/Lightbox.module.css, updated mobile rules for .close and .nav to maintain 44x44px touch targets on phones (dropping the 40px reduction).
  • Added min-height: 44px to .infoToggle buttons in the bottom bar on mobile.
  • Added :global(body):has(> .overlay) :global(main) { visibility: hidden; } so underlying album titles, back links, and photo wall tiles do not bleed through the translucent overlay.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant