Merge pull request 'fix(detail): label the phase of all-through admissions figures (Batch F review fix)' (#69) from fix/detail-allthrough-admissions-phase into main

Reviewed-on: #69

nextjs-app/components/SchoolDetailView.module.css

@@ -1040,41 +1040,6 @@
     color: var(--accent-coral-dark, #b04a2e);
 }
 
-/* ── Ofsted grade grids (Report Card + OEIF) ──
-   Uniform, vertically-aligned grade chips. Labels reserve two lines so
-   single- and double-line labels put their chips on the same baseline;
-   every chip (Met, Strong, Expected standard, …) shares one font size,
-   padding and min-height regardless of how many lines its text wraps to. */
-.gradeGrid .metricCard {
-    display: flex;
-    flex-direction: column;
-    align-items: center;
-    gap: 0.5rem;
-    padding: 0.85rem 0.75rem;
-}
-.gradeGrid .metricLabel {
-    min-height: 2.6em;
-    margin: 0;
-    display: flex;
-    align-items: center;
-    justify-content: center;
-    text-align: center;
-}
-.gradeGrid .metricValue {
-    margin-top: auto;
-    display: inline-flex;
-    align-items: center;
-    justify-content: center;
-    max-width: 100%;
-    min-height: 2.6em;
-    padding: 0.3rem 0.7rem;
-    border-radius: 5px;
-    font-size: 1rem;
-    font-weight: 700;
-    line-height: 1.25;
-    text-align: center;
-}
-
 .ofstedDisclaimer {
     font-size: 0.8rem;
     color: var(--text-muted, #8a847a);

nextjs-app/components/SchoolDetailView.tsx

@@ -555,7 +555,7 @@ export function SchoolDetailView({
               <p className={styles.ofstedDisclaimer}>
                 From November 2025, Ofsted replaced single overall grades with Report Cards rating schools across several areas.
               </p>
-              <div className={`${styles.metricsGrid} ${styles.gradeGrid}`}>
+              <div className={styles.metricsGrid}>
                 {ofsted.rc_safeguarding_met != null && (
                   <div className={styles.metricCard}>
                     <div className={styles.metricLabel}>Safeguarding</div>
@@ -601,7 +601,7 @@ export function SchoolDetailView({
                   Rated <strong>{OFSTED_LABELS[ofsted.overall_effectiveness!]}</strong> across all inspected areas — Quality of Teaching, Behaviour, Pupils&apos; Development and Leadership.
                 </p>
               ) : (
-                <div className={`${styles.metricsGrid} ${styles.gradeGrid}`}>
+                <div className={styles.metricsGrid}>
                   {oeifAreas.map(({ label, value }) => (
                     <div key={label} className={styles.metricCard}>
                       <div className={styles.metricLabel}>{label}</div>
@@ -887,6 +887,15 @@ export function SchoolDetailView({
               </div>
             )}
           </div>
+          {/* All-through admissions data covers a single entry point (usually the
+              Year 7 secondary intake), not reception — say so, or a parent could
+              read these as the whole-school figures. */}
+          {isAllThrough && admissions.school_phase && (
+            <p className={styles.sectionSubtitle}>
+              These figures are for {admissions.school_phase.toLowerCase()} entry
+              {/secondary/i.test(admissions.school_phase) ? ' (Year 7)' : /primary/i.test(admissions.school_phase) ? ' (Reception)' : ''}.
+            </p>
+          )}
 
           <div className={styles.admissionsViewport}>
             {/* This-year Q&A */}

nextjs-app/components/SecondarySchoolDetailView.module.css

@@ -623,41 +623,6 @@
     color: var(--accent-coral-dark, #b04a2e);
 }
 
-/* ── Ofsted grade grids (Report Card + OEIF) ──
-   Uniform, vertically-aligned grade chips. Labels reserve two lines so
-   single- and double-line labels put their chips on the same baseline;
-   every chip (Met, Strong, Expected standard, …) shares one font size,
-   padding and min-height regardless of how many lines its text wraps to. */
-.gradeGrid .metricCard {
-    display: flex;
-    flex-direction: column;
-    align-items: center;
-    gap: 0.5rem;
-    padding: 0.85rem 0.75rem;
-}
-.gradeGrid .metricLabel {
-    min-height: 2.6em;
-    margin: 0;
-    display: flex;
-    align-items: center;
-    justify-content: center;
-    text-align: center;
-}
-.gradeGrid .metricValue {
-    margin-top: auto;
-    display: inline-flex;
-    align-items: center;
-    justify-content: center;
-    max-width: 100%;
-    min-height: 2.6em;
-    padding: 0.3rem 0.7rem;
-    border-radius: 5px;
-    font-size: 1rem;
-    font-weight: 700;
-    line-height: 1.25;
-    text-align: center;
-}
-
 .ofstedDisclaimer {
     font-size: 0.8rem;
     color: var(--text-muted, #8a847a);

nextjs-app/components/SecondarySchoolDetailView.tsx

@@ -363,7 +363,7 @@ export function SecondarySchoolDetailView({
               <p className={styles.ofstedDisclaimer}>
                 From November 2025, Ofsted replaced single overall grades with Report Cards rating schools across several areas.
               </p>
-              <div className={`${styles.metricsGrid} ${styles.gradeGrid}`}>
+              <div className={styles.metricsGrid}>
                 {ofsted.rc_safeguarding_met != null && (
                   <div className={styles.metricCard}>
                     <div className={styles.metricLabel}>Safeguarding</div>
@@ -408,7 +408,7 @@ export function SecondarySchoolDetailView({
                   Rated <strong>{OFSTED_LABELS[ofsted.overall_effectiveness]}</strong> across all inspected areas — Quality of Teaching, Behaviour, Pupils&apos; Development and Leadership.
                 </p>
               ) : (
-                <div className={`${styles.metricsGrid} ${styles.gradeGrid}`}>
+                <div className={styles.metricsGrid}>
                   {oeifAreas.map(({ label, value }) => (
                     <div key={label} className={styles.metricCard}>
                       <div className={styles.metricLabel}>{label}</div>
@@ -425,7 +425,7 @@ export function SecondarySchoolDetailView({
               <p className={styles.sectionSubtitle}>
                 From September 2024, Ofsted no longer gives a single overall grade.
               </p>
-              <div className={`${styles.metricsGrid} ${styles.gradeGrid}`}>
+              <div className={styles.metricsGrid}>
                 {[
                   { label: 'Quality of Education', value: ofsted.quality_of_education },
                   { label: 'Behaviour & Attitudes', value: ofsted.behaviour_attitudes },