TudorandClaude Opus 5
8c3a5cc4e9
PR Checks / Frontend Typecheck + Tests (pull_request) Successful in 1m2s
PR Checks / Backend Smoke (pull_request) Successful in 7s
PR Checks / Build Backend (no push) (pull_request) Successful in 11s
PR Checks / Build Frontend (no push) (pull_request) Successful in 49s
PR Checks / Build Pipeline (no push) (pull_request) Successful in 10s
PR Checks / AI Code Review (Claude) (pull_request) Failing after 1m8s
Review follow-up on #86. The blind coral -> brand rename recreated the exact collision this PR set out to remove: coral had been both the primary CTA and the "below average" signal, so every negative indicator followed --primary onto iris. Sixteen rules moved back onto the status ramp — delta chips, trend-down arrows, progress-negative values, statusBad, chipBad/badgeBad, and the urgent deadline chips. The Ofsted scale had also lost its worst step, with grade 4 landing on brand while 1-2 were teal and 3 was amber. It now escalates by weight rather than by reaching for another hue: a tinted amber chip for "requires improvement", a solid amber one for "inadequate" (5.1:1 light, 7.7:1 dark). Report-card grade 5 follows the same rule. globals.css now describes status as valence — teal above/good, amber below/needs-attention — which is what it has to mean for an urgent deadline, rather than the narrower "comparison point only" the first draft claimed. On the share-card fonts: /opengraph-image is prerendered, so the font read happens in the builder stage where assets/ exists, and the baked PNG ships inside .next/standalone/.next/server/app/. File tracing independently places the fonts at .next/standalone/assets, which the existing standalone COPY carries to /app/assets. So the reported ENOENT doesn't occur — but it depends on the tracer resolving a runtime join(), and a miss would be a silent 500 rather than a build failure. Declared outputFileTracingIncludes for the route and made the Dockerfile COPY explicit so neither is left to inference. Also repointed the immutable Cache-Control rule from the deleted favicon.svg to app/icon.svg, where it was caching a 404. Verified: tsc clean, 159/159 tests, clean rebuild prerenders all three image routes with the fonts present in standalone. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
254 lines
4.8 KiB
CSS
254 lines
4.8 KiB
CSS
/* Shared layout for the compare screen's measure-first sections.
|
||
Mobile base: each row-label becomes a measure header and each school cell
|
||
stacks under it (colour-coded via the cell's ::before school tag).
|
||
Desktop (≥761px): the mockups' grid — 200px row-label column + one column
|
||
per school (2–4 columns supported via --school-count). */
|
||
|
||
.section {
|
||
margin-top: 3rem;
|
||
}
|
||
|
||
.sectionTitle {
|
||
font-family: var(--font-display);
|
||
font-size: 1.45rem;
|
||
font-weight: 700;
|
||
margin: 0;
|
||
padding-left: 0.75rem;
|
||
border-left: 3px solid var(--brand-strong);
|
||
}
|
||
|
||
.how {
|
||
font-size: 0.85rem;
|
||
color: var(--text-muted);
|
||
margin: 0.35rem 0 0 0.95rem;
|
||
max-width: 70ch;
|
||
}
|
||
|
||
.grid {
|
||
display: grid;
|
||
grid-template-columns: 1fr;
|
||
gap: 0;
|
||
margin-top: 1.25rem;
|
||
}
|
||
|
||
/* Mobile base: each measure is a card; each cell is a school row led by a
|
||
colour dot + short name. `display: contents` at ≥761px dissolves the card
|
||
back into the shared grid. */
|
||
.measure {
|
||
background: var(--bg-card);
|
||
border: 1px solid var(--border);
|
||
border-radius: 12px;
|
||
box-shadow: var(--shadow-soft);
|
||
padding: 0.75rem 0.85rem;
|
||
margin-bottom: 0.6rem;
|
||
}
|
||
|
||
.rowLabel {
|
||
font-size: 0.85rem;
|
||
font-weight: 600;
|
||
color: var(--text-primary);
|
||
display: flex;
|
||
align-items: center;
|
||
gap: 0.35rem;
|
||
padding: 0 0 0.1rem;
|
||
}
|
||
|
||
.cell {
|
||
display: flex;
|
||
align-items: baseline;
|
||
gap: 0.35rem 0.5rem;
|
||
flex-wrap: wrap;
|
||
padding: 0.5rem 0;
|
||
border-top: 1px solid var(--border);
|
||
margin-top: 0.5rem;
|
||
font-size: 0.95rem;
|
||
}
|
||
|
||
/* The school name gets its own full-width line above the value — real
|
||
school names are long and varied, so a fixed-width name column truncated
|
||
them ("Our Lady Queen of H…") or crowded the value. */
|
||
.cellTag {
|
||
display: inline-flex;
|
||
align-items: center;
|
||
gap: 0.4rem;
|
||
flex-basis: 100%;
|
||
/* Slightly larger than the values below it so the school each row belongs
|
||
to is easy to read on mobile (hidden on desktop, where the column header
|
||
names the school). */
|
||
font-size: 0.9rem;
|
||
font-weight: 600;
|
||
color: var(--sc, var(--text-secondary));
|
||
margin-bottom: 0.2rem;
|
||
}
|
||
|
||
.cellDot {
|
||
width: 10px;
|
||
height: 10px;
|
||
border-radius: 50%;
|
||
background: var(--dot, var(--text-muted));
|
||
flex: none;
|
||
}
|
||
|
||
.big {
|
||
font-size: 1.05rem;
|
||
font-weight: 700;
|
||
font-variant-numeric: tabular-nums;
|
||
}
|
||
|
||
.small {
|
||
display: block;
|
||
flex-basis: 100%;
|
||
font-size: 0.8rem;
|
||
color: var(--text-muted);
|
||
margin-top: 0;
|
||
}
|
||
|
||
.chip {
|
||
display: inline-block;
|
||
font-size: 0.75rem;
|
||
font-weight: 600;
|
||
border-radius: 999px;
|
||
padding: 0.15rem 0.6rem;
|
||
white-space: nowrap;
|
||
}
|
||
|
||
.chipGood {
|
||
background: rgba(var(--status-above-rgb), 0.14);
|
||
color: var(--status-above);
|
||
}
|
||
|
||
.chipWarn {
|
||
background: var(--status-below-bg);
|
||
color: var(--status-below);
|
||
}
|
||
|
||
.chipBad {
|
||
background: var(--status-below-bg);
|
||
color: var(--status-below);
|
||
}
|
||
|
||
.chipNeutral {
|
||
background: var(--bg-secondary);
|
||
color: var(--text-secondary);
|
||
}
|
||
|
||
.badge {
|
||
display: inline-block;
|
||
font-weight: 700;
|
||
border-radius: 6px;
|
||
padding: 0.25rem 0.7rem;
|
||
font-size: 0.9rem;
|
||
}
|
||
|
||
.badgeGood {
|
||
background: rgba(var(--status-above-rgb), 0.14);
|
||
color: var(--status-above);
|
||
}
|
||
|
||
.badgeWarn {
|
||
background: var(--status-below-bg);
|
||
color: var(--status-below);
|
||
}
|
||
|
||
.badgeBad {
|
||
background: var(--status-below-bg);
|
||
color: var(--status-below);
|
||
}
|
||
|
||
.rcList {
|
||
display: flex;
|
||
flex-direction: column;
|
||
gap: 0.3rem;
|
||
margin-top: 0.2rem;
|
||
}
|
||
|
||
.rcRow {
|
||
display: flex;
|
||
justify-content: space-between;
|
||
align-items: center;
|
||
gap: 0.5rem;
|
||
font-size: 0.8rem;
|
||
}
|
||
|
||
.rcArea {
|
||
color: var(--text-secondary);
|
||
}
|
||
|
||
.chipStack {
|
||
display: flex;
|
||
gap: 0.3rem;
|
||
flex-wrap: wrap;
|
||
flex-basis: 100%;
|
||
margin-top: 0.3rem;
|
||
}
|
||
|
||
.barMini {
|
||
display: block;
|
||
height: 8px;
|
||
border-radius: 4px;
|
||
background: var(--bg-secondary);
|
||
overflow: hidden;
|
||
margin-top: 0.3rem;
|
||
max-width: 140px;
|
||
}
|
||
|
||
.barMini > i {
|
||
display: block;
|
||
height: 100%;
|
||
border-radius: 4px;
|
||
}
|
||
|
||
.card {
|
||
background: var(--bg-card);
|
||
border: 1px solid var(--border);
|
||
border-radius: 16px;
|
||
box-shadow: var(--shadow-soft);
|
||
padding: 1.25rem 1.5rem;
|
||
margin-top: 1rem;
|
||
}
|
||
|
||
.link {
|
||
color: var(--brand-strong);
|
||
}
|
||
|
||
@media (min-width: 761px) {
|
||
.grid {
|
||
grid-template-columns: 200px repeat(var(--school-count, 3), 1fr);
|
||
gap: 0 0.75rem;
|
||
}
|
||
|
||
/* Dissolve the per-measure card so its label + cells become grid items of
|
||
.grid, keeping columns aligned across every measure. */
|
||
.measure {
|
||
display: contents;
|
||
}
|
||
|
||
.cellTag {
|
||
display: none;
|
||
}
|
||
|
||
.rowLabel {
|
||
color: var(--text-secondary);
|
||
padding: 0.85rem 0.5rem 0.85rem 0;
|
||
border-bottom: 1px solid var(--border);
|
||
}
|
||
|
||
.cell {
|
||
display: block;
|
||
padding: 0.85rem 0.25rem;
|
||
border-top: none;
|
||
border-bottom: 1px solid var(--border);
|
||
margin-top: 0;
|
||
}
|
||
|
||
.big {
|
||
font-size: 1.35rem;
|
||
}
|
||
|
||
.small {
|
||
flex-basis: auto;
|
||
padding-left: 0;
|
||
margin-top: 0.1rem;
|
||
}
|
||
}
|