Skip to content

Fix verify page exhale-note dialog visibility on mobile#342

Merged
kojibai merged 1 commit intomainfrom
codex/fix-exhale-note-popover-on-mobile
Jan 31, 2026
Merged

Fix verify page exhale-note dialog visibility on mobile#342
kojibai merged 1 commit intomainfrom
codex/fix-exhale-note-popover-on-mobile

Conversation

@kojibai
Copy link
Copy Markdown
Owner

@kojibai kojibai commented Jan 31, 2026

Motivation

  • Mobile browsers were showing a blank/crystallized screen when opening the exhale-note popover on the verify page because the dialog was not getting a visible/fixed rendering state on small viewports.

Description

  • Updated src/pages/VerifyPage.css to explicitly hide dialog.glass-modal by default and show it when the dialog is open via dialog.glass-modal[open] to ensure predictable rendering.
  • Added positioning for dialog.glass-modal.fullscreen[open] to use position: fixed with a safe inset and high z-index so the fullscreen dialog is anchored correctly on mobile and desktop.
  • Added a media query to account for safe-area insets on small screens using var(--safeTop|Right|Bot|Left) so the dialog respects device notches and bottom bars.

Testing

  • Launched the dev server with npm run dev -- --host 0.0.0.0 --port 4173 and confirmed Vite served the app successfully.
  • Ran a Playwright script that loaded http://127.0.0.1:4173/verify at a mobile viewport (390x844), forced the note dialog open attribute, and captured a screenshot to artifacts/verify-note-mobile.png, which validated the dialog is visible and positioned as expected.
  • Committed the change with message Fix verify note dialog visibility on mobile and created this PR.

Codex Task

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
phi-network-5aii Ready Ready Preview, Comment Jan 31, 2026 6:06pm

Request Review

@kojibai kojibai merged commit 2c5d5a3 into main Jan 31, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant