Skip to content

Fix verify note layout and dialog controls#341

Merged
kojibai merged 2 commits intocodex/add-exhale-note-feature-to-verifypagefrom
codex/fix-note-button-functionality-on-verify-page
Jan 31, 2026
Merged

Fix verify note layout and dialog controls#341
kojibai merged 2 commits intocodex/add-exhale-note-feature-to-verifypagefrom
codex/fix-note-button-functionality-on-verify-page

Conversation

@kojibai
Copy link
Copy Markdown
Owner

@kojibai kojibai commented Jan 31, 2026

Motivation

  • Improve the Verify page UX by separating Exhale note controls from the proof action buttons and make the note popover/dialog behave predictably and be dismissible.

Description

  • Reorganized the Verify header to group proof actions into a vreceipt-block and moved Exhale note controls into a new vreceipt-note section beneath it in src/pages/VerifyPage.tsx.
  • Made the note-open button use onPointerDown (and fallback onClick) so the note dialog opens on the first interaction instead of requiring a double click in src/pages/VerifyPage.tsx.
  • Added CSS rules for the new layout (.vreceipt-block, .vreceipt-note, etc.) and introduced dialog.glass-modal + .modal-topbar .close-btn styling so the note dialog and its close “×” button are visible and accessible in src/pages/VerifyPage.css.
  • Minor tidy-up of button ordering and handlers for share/copy/sign/download actions so proof controls remain grouped while note controls are isolated.

Testing

  • Launched the dev server with npm run dev and confirmed it started successfully (Vite listening at localhost:4173). — succeeded.
  • Captured a UI screenshot via a Playwright script navigating to /verify to verify layout and dialog appearance; screenshot artifact was produced (artifacts/verify-note-section.png). — succeeded.
  • No unit or integration test changes were required for this UI-only change. — N/A.

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 5:37pm

Request Review

@kojibai kojibai merged commit 6a2108d into codex/add-exhale-note-feature-to-verifypage Jan 31, 2026
1 of 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