Skip to content

ui: mobile hamburger menu, nav rename, honest provenance copy - #14

Merged
saquibreja7-hash merged 1 commit into
masterfrom
ui/mobile-menu-and-copy
Aug 2, 2026
Merged

ui: mobile hamburger menu, nav rename, honest provenance copy#14
saquibreja7-hash merged 1 commit into
masterfrom
ui/mobile-menu-and-copy

Conversation

@saquibreja7-hash

Copy link
Copy Markdown
Owner

Changes

  • Mobile menu fix: replaced the flaky native <details> menu (its <summary> toggle was suppressed by the button styling) with a real client-side MobileMenu component. Reliable tap toggle on mobile, closes on link tap / outside tap / Escape, accessible (aria-expanded/aria-controls). Uses a hamburger icon that becomes an X when open.
  • Nav rename: 'Check image' → 'Check AI / Deepfake' (EN + HI).
  • Honest copy on /check-image: removed the 'Runs entirely in your browser' badge and reworded the hero subtitle + privacy note, since the OpenAI deeper check does upload. They now say the first check runs on-device and nothing is sent anywhere unless the user chooses the deeper check.
  • Docs: fixed a stale .env.example comment that wrongly named Azure; the database is Neon.

Verification

  • type-check clean, JSON valid, lint clean (pre-existing warnings only).

🤖 Generated with Claude Code

- Replace the flaky native <details> mobile menu with a real client-side
  MobileMenu (button + state); reliable toggle on mobile, closes on link tap,
  outside tap, and Escape; accessible aria-expanded/controls. Uses a hamburger
  icon that becomes an X when open.
- Rename the nav item 'Check image' to 'Check AI / Deepfake' (EN + HI).
- Remove the 'Runs entirely in your browser' badge and reword the hero subtitle
  and privacy note on /check-image: they now say the first check runs on-device
  and nothing is sent anywhere unless the user chooses the OpenAI deeper check.
- Fix a stale .env.example comment that wrongly named Azure; the database is Neon.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 2, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
meri-asmita Ready Ready Preview Aug 2, 2026 3:06pm

@claude

claude Bot commented Aug 2, 2026

Copy link
Copy Markdown

Review summary

Scope: .env.example (Azure→Neon connection-string docs), mobile hamburger menu component (MobileMenu.tsx replacing the <details> nav), nav label rename ("Check image" → "Check AI / Deepfake"), and copy updates on /check-image (hero sub + privacy note) in en/hi.

Safety invariants: No code paths here touch URL/media handling, Aadhaar, notice dispatch, hash submission, or the ENABLE_HASH_UPLOAD/ENABLE_PLATFORM_API gates. The copy change removes the "never uploaded"/"runs entirely in your browser" claims and replaces them with accurate wording ("only sent if you choose the deeper check") — verified against the existing check.deep.* flow in ImageChecker.tsx and /api/check-image/route.ts, which is gated behind the off-by-default ENABLE_PROVENANCE_CHECK flag, requires explicit consent + age confirmation, and is documented in ADR 002. This PR doesn't change that route; it just makes the front-end copy match what the code already does, which is an improvement (previously the copy was inaccurate once the deep-check feature shipped).

Security/business-logic/data-integrity: Nothing in this diff touches auth, CSRF, rate limiting, encryption, case lifecycle, notice/hash dispatch, or the audit chain. MobileMenu.tsx is a client-only nav affordance (open/close state, outside-click/Escape handlers) with no data fetching or side effects of note.

Minor (not blocking): the now-unused check.privacy.badge i18n key is left in both en.json/hi.json after the badge <div> was removed from check-image/page.tsx — harmless dead string, not flagging as a line comment since it's cleanup/style.

No CRITICAL or WARNING findings within the review's focus areas.

Verdict: PASS

@saquibreja7-hash
saquibreja7-hash merged commit d12c01c into master Aug 2, 2026
8 checks passed
@saquibreja7-hash
saquibreja7-hash deleted the ui/mobile-menu-and-copy branch August 2, 2026 15:07
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