Skip to content

Minimize analytics consent modal - #150

Draft
feynon wants to merge 1 commit into
mainfrom
agent/minimize-analytics-consent-modal
Draft

Minimize analytics consent modal#150
feynon wants to merge 1 commit into
mainfrom
agent/minimize-analytics-consent-modal

Conversation

@feynon

@feynon feynon commented Aug 15, 2026

Copy link
Copy Markdown
Member

What changed

  • Reduced the desktop modal width from 36rem to 28rem.
  • Tightened mobile and desktop padding, corner radius, paragraph spacing, and line height.
  • Reduced the body copy to 14px and the actions to compact 80 × 40px controls with 12px labels.
  • Kept all consent copy, analytics behavior, focus handling, safe-area positioning, and accessibility attributes unchanged.

Why

The consent modal occupied too much of the mobile viewport and felt oversized on desktop. Its large width, padding, line height, and controls made a short consent decision visually dominate the page.

Impact

The modal now reads as a compact, unobtrusive consent prompt at both mobile and desktop sizes while remaining fully usable and scroll-safe within 100dvh.

Validation

  • git diff --check
  • Rendered /book successfully in Next.js dev mode at 390 × 844 and 1440 × 900 viewports
  • Visually confirmed the modal at both viewports with no blank page or visible framework error overlay
  • npx tsc --noEmit reaches this component but remains blocked by existing unrelated type errors elsewhere in the repository
  • The repository lint script cannot currently run because ESLint and an ESLint config are not installed

@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
tiles-next Ready Ready Preview, v0 Aug 15, 2026 6:00pm

Request Review

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