style(compare): shorten Ofsted link text to 'Ofsted page'

The link repeated the full school name ('Watford Grammar School for Boys's Ofsted page') — long on mobile, and the colour-coded school name already leads the row. Show just 'Ofsted page →'; keep the full name in aria-label so screen readers can still tell the per-school links apart. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0146VHeLAWjDVE2B5uU67jCB
Merge pull request 'style(compare): enlarge per-school name label in mobile section cards' (#59 ) from fix/mobile-cell-name-size into main
2026-07-18 15:19:41 +01:00 · 2026-07-18 14:09:02 +00:00
1 changed files with 11 additions and 2 deletions
@@ -239,8 +239,17 @@ export function CompareOfsted({
            `https://reports.ofsted.gov.uk/provider/21/${school.urn}`;
          return (
            <Cell key={school.urn} school={school} index={i}>
-              <a className={s.link} href={url} target="_blank" rel="noopener noreferrer">
-                {school.school_name}&apos;s Ofsted page →
+              {/* Short visible label to save space on mobile (the coloured
+                  school name already leads the row); the full name stays in
+                  aria-label so screen readers can tell the links apart. */}
+              <a
+                className={s.link}
+                href={url}
+                target="_blank"
+                rel="noopener noreferrer"
+                aria-label={`${school.school_name}'s Ofsted page`}
+              >
+                Ofsted page →
              </a>
            </Cell>
          );
Author	SHA1	Message	Date
TudorandClaude Fable 5	3e77057567	style(compare): shorten Ofsted link text to 'Ofsted page' PR Checks / Frontend Typecheck + Tests (pull_request) Successful in 1m5s Details PR Checks / Backend Smoke (pull_request) Successful in 7s Details PR Checks / Build Backend (no push) (pull_request) Successful in 10s Details PR Checks / Build Frontend (no push) (pull_request) Successful in 45s Details PR Checks / Build Pipeline (no push) (pull_request) Successful in 10s Details PR Checks / AI Code Review (Claude) (pull_request) Successful in 9s Details The link repeated the full school name ('Watford Grammar School for Boys's Ofsted page') — long on mobile, and the colour-coded school name already leads the row. Show just 'Ofsted page →'; keep the full name in aria-label so screen readers can still tell the per-school links apart. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0146VHeLAWjDVE2B5uU67jCB	2026-07-18 15:19:41 +01:00
tudor	e0d5a9969e	Merge pull request 'style(compare): enlarge per-school name label in mobile section cards' (#59 ) from fix/mobile-cell-name-size into main Stage (build -> staging -> E2E gate) / Build Backend (FastAPI) (push) Successful in 14s Details Stage (build -> staging -> E2E gate) / Build Frontend (Next.js) (push) Successful in 49s Details Stage (build -> staging -> E2E gate) / Build Pipeline (Meltano + dbt + Airflow) (push) Successful in 14s Details Stage (build -> staging -> E2E gate) / Deploy to Staging (push) Successful in 1s Details Stage (build -> staging -> E2E gate) / E2E Journeys against Staging (push) Successful in 41s Details Reviewed-on: #59	2026-07-18 14:09:02 +00:00