Repository navigation
Conversation
Full-route audit at 390px found no horizontal overflow and compliant tap targets almost everywhere — the only misses were three standalone text links rendering ~17-20px tall: - homepage "View all" link in the Our Beers teaser header - beer-detail breadcrumb "Our Beers" - beers page "Submit a trade inquiry" link in the For Partners panel All three now render as 44px-tall tappable rows, matching the established inline-flex min-h-[44px] pattern used by venue cards, footer links, and contact actions. Prose-internal links (privacy, terms, trade email footer) are inline content and exempt per WCAG 2.5.8; they are deliberately unchanged. Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR implements the only issues found in a 390px mobile audit: three standalone links now render as vertically centered 44px rows via Tailwind utility classes. The broader audit found no overflow or interaction issues, and intentionally leaves safe-area handling, loading boundaries, and menu/dialog behavior unchanged; full automated validation remains pending on the stacked branch. File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Summary
The deliverable from the full mobile-first audit: a 390px sweep across every public route on the integrated polish stack found the site in good shape — this PR applies the only fixes that surfaced, three standalone links below the 44px tap-target floor.
Stacked on
feat/polish-integration(the reviewable merge of #188 + #189 + #190). Retargets tomainonce the stack merges.Closes #168
Audit method
Programmatic probe at 390×844 with touch emulation across
/,/beers,/beers/neipa,/where-to-buy,/about,/contact,/trade,/pay/complete,/pay/cancelled,/privacy,/terms,/trade/login, and the 404 boundary, plus manual Playwright review of the mobile menu, tour dialog, Klaro notice + preferences modal, venue cards, carousel, and footer.Findings — clean
scrollWidth == clientWidtheverywhere).Fixes
Three standalone links rendered ~17-20px tall — now 44px rows using the established
inline-flex min-h-[44px] items-centerpattern:View all →(Our Beers teaser header)Our BeersSubmit a trade inquiry →(For Partners panel)Deliberate non-changes
env(safe-area-inset-*)on the pill nav — itstop-4offset sits below the status-bar cutout in portrait, and landscape notches overlap the side edges, not the top. Verified unnecessary.loading.tsxwould essentially never trigger.Verification
Generated with Devin
Summary by Sourcery
Bug Fixes: