School pages no longer print a sentence under the "Other schools nearby" heading. It read "Other primary schools near .", which repeated the heading.
Changes
NearbySchoolsSection: the heading is now the whole header. nearbyNoun() and the phase and schoolName props existed only to build the sentence, so they are removed, along with their tests and the props at both call sites.
NearbySchools.module.css: the sentence's 1.25rem bottom margin was the only gap between the heading and the cards, so it moves to .top. With only the heading left in that row, .top now centres it against the carousel arrows instead of aligning to their top.
Comments that described the sentence are updated, and an old one about intake claims that no longer referred to anything is removed.
Verification
npm test: 72 suites, 621 tests pass.
npm run typecheck: clean.
Staging preview, made by removing the sentence in the browser and applying the same CSS on Allfarthing Primary School, not a deploy. At 1280px the cards sit 18px below the heading (the same gap as before) and the heading is centred on the arrows. At 390px nothing spills past the right edge.
No e2e journey checks the sentence, so none change. There is no new test that it stays gone, because that would only check wording.
## What
School pages no longer print a sentence under the "Other schools nearby" heading. It read "Other primary schools near <school>.", which repeated the heading.
## Changes
- `NearbySchoolsSection`: the heading is now the whole header. `nearbyNoun()` and the `phase` and `schoolName` props existed only to build the sentence, so they are removed, along with their tests and the props at both call sites.
- `NearbySchools.module.css`: the sentence's 1.25rem bottom margin was the only gap between the heading and the cards, so it moves to `.top`. With only the heading left in that row, `.top` now centres it against the carousel arrows instead of aligning to their top.
- Comments that described the sentence are updated, and an old one about intake claims that no longer referred to anything is removed.
## Verification
- `npm test`: 72 suites, 621 tests pass.
- `npm run typecheck`: clean.
- Staging preview, made by removing the sentence in the browser and applying the same CSS on Allfarthing Primary School, not a deploy. At 1280px the cards sit 18px below the heading (the same gap as before) and the heading is centred on the arrows. At 390px nothing spills past the right edge.
No e2e journey checks the sentence, so none change. There is no new test that it stays gone, because that would only check wording.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
"Other schools nearby" was followed by "Other primary schools near
<school>.", which says the same thing again. The heading now stands alone.
nearbyNoun() and the phase and schoolName props existed only to build that
line, so they go with it. Its bottom margin was the only gap between the
heading and the cards, so the header row carries that gap now, and centres
the heading against the carousel arrows now that it is a single line.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Removes the lede paragraph and the nearbyNoun helper from the nearby-schools section, along with the now-unused schoolName and phase props, and updates callers, CSS and tests. This is a consistent UI-only cleanup with no data or backend impact, and it looks healthy.
✅ No issues found.
## 🤖 AI Code Review (Claude Code)
Removes the lede paragraph and the nearbyNoun helper from the nearby-schools section, along with the now-unused schoolName and phase props, and updates callers, CSS and tests. This is a consistent UI-only cleanup with no data or backend impact, and it looks healthy.
✅ No issues found.
tudor
merged commit e1373fb6df into main2026-10-04 09:13:48 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
What
School pages no longer print a sentence under the "Other schools nearby" heading. It read "Other primary schools near .", which repeated the heading.
Changes
NearbySchoolsSection: the heading is now the whole header.nearbyNoun()and thephaseandschoolNameprops existed only to build the sentence, so they are removed, along with their tests and the props at both call sites.NearbySchools.module.css: the sentence's 1.25rem bottom margin was the only gap between the heading and the cards, so it moves to.top. With only the heading left in that row,.topnow centres it against the carousel arrows instead of aligning to their top.Verification
npm test: 72 suites, 621 tests pass.npm run typecheck: clean.No e2e journey checks the sentence, so none change. There is no new test that it stays gone, because that would only check wording.
🤖 Generated with Claude Code
🤖 AI Code Review (Claude Code)
Removes the lede paragraph and the nearbyNoun helper from the nearby-schools section, along with the now-unused schoolName and phase props, and updates callers, CSS and tests. This is a consistent UI-only cleanup with no data or backend impact, and it looks healthy.
✅ No issues found.