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

fix(ui): render safeguarding as a standard metric card for visual consistency
All checks were successful
Build and Push Docker Images / Build Backend (FastAPI) (push) Successful in 32s
Details
Build and Push Docker Images / Build Frontend (Next.js) (push) Successful in 1m10s
Details
Build and Push Docker Images / Build Integrator (push) Successful in 56s
Details
Build and Push Docker Images / Build Kestra Init (push) Successful in 32s
Details
Build and Push Docker Images / Trigger Portainer Update (push) Successful in 1s
Details

Browse Source 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Tudor
2026-03-25 15:17:45 +00:00
parent d81f03cfcf
commit 5cdafc887e
2 changed files with 9 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
nextjs-app/components/SchoolDetailView.module.css
View File

@@ -482,18 +482,7 @@
.rcGrade4 { background: rgba(249, 115, 22, 0.12); color: #c2410c; } /* Needs attention */
.rcGrade5 { background: rgba(224, 114, 86, 0.15); color: var(--accent-coral, #e07256); } /* Urgent improvement */
/* Safeguarding badge */
.safeguardingRow {
display: flex;
align-items: center;
gap: 0.75rem;
margin-bottom: 1rem;
}
.safeguardingLabel {
font-size: 0.875rem;
font-weight: 600;
color: var(--text-primary, #1a1612);
}
/* Safeguarding value (used inside a standard metricCard) */
.safeguardingMet {
display: inline-block;
padding: 0.2rem 0.6rem;

16
nextjs-app/components/SchoolDetailView.tsx
View File

@@ -220,15 +220,15 @@ export function SchoolDetailView({
<p className={styles.ofstedDisclaimer}>
From November 2025, Ofsted replaced single overall grades with Report Cards rating schools across several areas.
</p>
{ofsted.rc_safeguarding_met != null && (
<div className={styles.safeguardingRow}>
<span className={styles.safeguardingLabel}>Safeguarding</span>
<span className={ofsted.rc_safeguarding_met ? styles.safeguardingMet : styles.safeguardingNotMet}>
{ofsted.rc_safeguarding_met ? 'Met' : 'Not met'}
</span>
</div>
)}
<div className={styles.metricsGrid}>
{ofsted.rc_safeguarding_met != null && (
<div className={styles.metricCard}>
<div className={styles.metricLabel}>Safeguarding</div>
<div className={`${styles.metricValue} ${ofsted.rc_safeguarding_met ? styles.safeguardingMet : styles.safeguardingNotMet}`}>
{ofsted.rc_safeguarding_met ? 'Met' : 'Not met'}
</div>
</div>
)}
{RC_CATEGORIES.map(({ key, label }) => {
const value = ofsted[key] as number | null;
return value != null ? (