From 0fae5aadf49c3ad652e8384063ee8bb4313dd18c Mon Sep 17 00:00:00 2001 From: Nish <257724087+nish3451@users.noreply.github.com> Date: Sat, 15 Aug 2026 14:22:16 +0530 Subject: [PATCH] =?UTF-8?q?docs(lane1):=20reverify=20product-hero=20early-?= =?UTF-8?q?access=20CTA=20item=20=E2=80=94=20already=20fixed=20on=20main?= =?UTF-8?q?=20(PR=20#147)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Promptly and Drishti mobile heroes buried the Get early access CTA below the first viewport because the shared clamp(3.4rem, 7vw, 6.7rem) H1 with a 9ch cap made the long hero H1s ~544px tall and the action-row sat after the page-lead. Fixed in 2796b33 (PR #147): mobile-only .page-hero-card h1 clamp(2.1rem, 5.6vw, 3.4rem) with max-width: none, and the hero .action-row moved above .page-lead on both pages. Re-verified against fresh origin/main: the layout probe in scripts/test-public-heading-hierarchy.mjs passes (93 checks, 0 failures), and headless-Chromium measurements at 320/390/414 widths show the CTA at 469-584px — fully in-fold — on both pages (before: ~1035-1164px). No source change is possible or needed; this report documents the verification. Co-authored-by: CommandCodeBot --- ...hero-early-access-cta-reverify-20260815.md | 55 +++++++++++++++++++ 1 file changed, 55 insertions(+) create mode 100644 .lane/reports/docs-lane1-product-hero-early-access-cta-reverify-20260815.md diff --git a/.lane/reports/docs-lane1-product-hero-early-access-cta-reverify-20260815.md b/.lane/reports/docs-lane1-product-hero-early-access-cta-reverify-20260815.md new file mode 100644 index 00000000..fe6dd199 --- /dev/null +++ b/.lane/reports/docs-lane1-product-hero-early-access-cta-reverify-20260815.md @@ -0,0 +1,55 @@ +# Lane 1 — Product hero "Get early access" CTA stays in the first mobile viewport (reverify) + +## Item + +- [unreviewed-by-opus] Promptly and Drishti mobile heroes bury the newly added Get early access CTA below the first [viewport] + +## Verdict + +**Already fixed and merged on main (PR #147).** No code change is possible or +needed; this run re-verified the shipped fix against fresh `origin/main` and +the rendered mobile layout. + +## Evidence + +- Fix commit `2796b33` ("fix(public): keep product hero early-access CTA in + the first mobile viewport (#147)") is an ancestor of fresh `origin/main` + (`a6e1be8`): `git merge-base --is-ancestor 2796b33 origin/main` → true. +- Source checks on `origin/main`: + - `public/promptly/index.html` and `public/drishti/index.html` place the + hero `.action-row` (with the `Get early access` mailto button) directly + after the H1 and above `.page-lead`. + - `public/styles.css` (mobile `@media (max-width: 720px)` block) shrinks + the long hero H1s: `.page-hero-card h1 { font-size: clamp(2.1rem, 5.6vw, + 3.4rem); max-width: none; }`, so the heading no longer consumes ~544px. +- Enforcement suite `scripts/test-public-heading-hierarchy.mjs` asserts, in + its Playwright layout probe, that the early-access CTA is fully within the + first viewport at 320 and 390 on `/promptly/` and `/drishti/`, plus that + CTA copy is unchanged. It fails closed when Playwright is absent. +- Live on this checkout (branch off fresh `origin/main`): + - `node scripts/test-public-heading-hierarchy.mjs` → 93 checks, 0 + failures, including the CTA viewport assertions for both pages. + - Headless-Chromium measurements (isMobile, local static `public/`): + + | page | viewport | CTA top–bottom | in-fold | + |---|---|---|---| + | /promptly/ | 320×667 | 536–584px | yes | + | /promptly/ | 320×844 | 536–584px | yes | + | /promptly/ | 390×844 | 469–517px | yes | + | /promptly/ | 414×844 | 469–517px | yes | + | /drishti/ | 320×667 | 502–550px | yes | + | /drishti/ | 320×844 | 502–550px | yes | + | /drishti/ | 390×844 | 469–517px | yes | + | /drishti/ | 414×844 | 469–517px | yes | + + Before the fix the same probe measured Promptly CTA bottom ~1083px and + Drishti ~1164px at 390 — the CTA sat below the fold on every mobile + size. The hero H1 now ends at ~470–504px at 320 instead of ~544px. + +## Files + +None changed (verification-only run). The prior fix (PR #147) touched: +- `public/styles.css` +- `public/promptly/index.html` +- `public/drishti/index.html` +- `scripts/test-public-heading-hierarchy.mjs`