tudor/school_compare
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request 'style(compare): shorten Ofsted link text to 'Ofsted page'' (#60) from fix/ofsted-link-text into main
Stage (build -> staging -> E2E gate) / Build Backend (FastAPI) (push) Successful in 13s
Details
Stage (build -> staging -> E2E gate) / Build Frontend (Next.js) (push) Successful in 52s
Details
Stage (build -> staging -> E2E gate) / Build Pipeline (Meltano + dbt + Airflow) (push) Successful in 13s
Details
Stage (build -> staging -> E2E gate) / Deploy to Staging (push) Successful in 0s
Details
Stage (build -> staging -> E2E gate) / E2E Journeys against Staging (push) Successful in 41s
Details

Browse Source 
Reviewed-on: #60
This commit was merged in pull request #60.
This commit is contained in:
tudor
2026-07-18 14:20:09 +00:00
parent e0d5a9969e 3e77057567
commit 2ac26acf91
1 changed files with 11 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nextjs-app/components/compare/CompareOfsted.tsx
+11 -2
View File
@@ -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>
);