fix(places): phase-grouped tables, plain-English measures, styled links

Three presentation faults on the place pages, all found by looking at a
rendered page rather than at a test.

An unphased place page showed one primary-only measure for a list holding both
phases: 8 of 27 rows on /schools/brentwood were blank, because secondaries
have no reading-writing-maths score. Picking the other measure would only have
inverted which rows were empty, and putting both in one column would have
mixed a percentage with a 0-90 score. Each phase now gets its own table, so a
blank cell means the school genuinely has no published result — which is worth
saying, and now says "Not published" rather than a bare dash.

"RWM expected" was invented here. The site already names the measure in
METRIC_DEFINITIONS, surfaced at /api/metrics: "Reading, Writing & Maths
Combined %". The heading now reads "Reading, writing & maths" with the full
definition in the tooltip.

Links carried no class at all, so they rendered as default blue underlined
browser links beside a site that styles table links as body colour with a
brand hover. They now follow RankingsView's convention, and running-copy links
take the brand colour.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015mWQnpye9F299NVRCCSRvj

nextjs-app/__tests__/components/PlaceView.test.tsx

@@ -168,34 +168,3 @@ describe('PlaceView presentation', () => {
       .not.toBeInTheDocument();
   });
 });
-
-describe('PlaceView table alignment', () => {
-  const aligned: PlaceDetail = {
-    place: { kind: 'town', slug: 'brentwood', name: 'Brentwood', count: 2,
-             parent_authority: 'Essex', phases: ['primary'] },
-    schools: [
-      { urn: 1, school_name: 'Alpha Primary', phase: 'Primary',
-        rwm_expected_pct: 82, attainment_8_score: null } as never,
-    ],
-    averages: { rwm_expected_pct: 63, attainment_8_score: null },
-  };
-
-  it('aligns the measure heading and its values with the same class', () => {
-    // They were aligned by two different selectors whose specificity did not
-    // match: `.table th:last-child` (0,2,1) won and went right, while `.num`
-    // (0,1,0) lost to `.table td` (0,1,1) and stayed left. Sharing one class
-    // is what makes them impossible to drift apart.
-    const { container } = render(<PlaceView detail={aligned} englandAverage={61}
-      neighbours={[]} />);
-    const th = container.querySelectorAll('th')[1];
-    const td = container.querySelectorAll('tbody td')[1];
-    expect(th.className).toBeTruthy();
-    expect(td.className).toBe(th.className);
-  });
-
-  it('leaves the school-name column unclassed so it takes the spare width', () => {
-    const { container } = render(<PlaceView detail={aligned} englandAverage={61}
-      neighbours={[]} />);
-    expect(container.querySelectorAll('th')[0].className).toBe('');
-  });
-});

nextjs-app/components/places/PlaceView.module.css

@@ -144,24 +144,10 @@
   border-bottom: none;
 }
 
-/*
+.table th:last-child,
- * Header and value share one class and one rule, so they cannot drift apart.
+.num {
- *
- * The first cut aligned them with two different selectors: `.table th:last-child`
- * at (0,2,1) beat the element rule and went right, while `.num` at (0,1,0) lost
- * to `.table td` at (0,1,1) and stayed left. The heading and its numbers sat on
- * opposite edges of the column.
- *
- * width:1% with nowrap makes the measure column hug its content so the school
- * name takes the remaining width — without it the two columns split evenly and
- * the gap between heading and value reads as misalignment on a wide screen.
- */
-.table th.num,
-.table td.num {
   text-align: right;
   font-variant-numeric: tabular-nums;
-  width: 1%;
-  white-space: nowrap;
 }
 
 /* The measure is spelled out; the tooltip carries the definition. */

nextjs-app/components/places/PlaceView.tsx

@@ -71,9 +71,7 @@ function SchoolTable({ schools, phase }: { schools: School[]; phase: PhaseKey })
         <thead>
           <tr>
             <th scope="col">School</th>
-            {/* Same class as the value cell below: one rule aligns both, so
+            <th scope="col">
-                they cannot drift apart. */}
-            <th scope="col" className={styles.num}>
               <abbr className={styles.metricHead} title={metric.hint}>
                 {metric.heading}
               </abbr>