Repository navigation
feat(ui): finish the GBA theme pass on the Data Custodian; add theme tests - #246
Merged
Merged
Conversation
Follow-up to #244. - Data Custodian admin dashboard: GBA-green action buttons and icons (the rainbow btn-success/info/danger/warning are gone) and white "stat" tiles in place of the coloured cards. Login icons and the error pages' "Try Again" button use the palette; the error icon size moves from inline styles to a theme class. Semantic colours stay: status badges, Delete/Revoke, warning and danger error icons, neutral Cancel/Go Back. - Footer moved out of the page container in 18 templates (and the leftover <hr> separators removed), so it is a full-width child of <body> pinned to the bottom of short pages. The two legacy customer pages (#245) are untouched; the theme keeps a full-bleed rule for them until #245. - application.css reduced to the rules still used that the theme does not cover (table header, focus outline, print). The old Material palette, radii, hover lifts, gradients and light footer were fighting the theme, so the theme drops most of its !important overrides and the --gb-* bridge. - Theme: white inputs and tables (Bootstrap fills them with the grey body colour), stat-tile and status-icon classes, page content grows so the footer sits at the bottom. - Retail customers list: the Delete button is btn-sm like View/Edit (btn-group-sm does not reach a button nested in a form). - Tests: GbaThemeResourcesTest (DC) and GbaLayoutPagesTest (TP, the first TP test with a full Spring context) check that the theme, logo and favicons are served anonymously (the logo byte-for-byte, so resource filtering cannot corrupt it), that pages render the GBA layout with SRI-pinned assets, and that the TP public pages are reachable without login. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Nov 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to #244: finishes the Green Button Alliance theme pass on the Data Custodian and adds tests that lock the theme in. The two legacy customer pages are deliberately untouched; their implementation is tracked in #245.
Restyle (decorative colours only)
gba-stattiles with a green rule and figure.btn-outline-primary; the 5 rem icon size moved from inlinestyle=to a theme class.Footer and layout
.containerand becomes a full-width child of<body>. The leftover<hr>separators are removed, and the content area grows so the footer sits at the bottom of short pages (checked on the Change Password page).CSS cleanup
application.cssis cut to the rules still in use that the theme doesn't cover: table header row, focus outline, print. The old Material palette, radii, hover lifts, gradients and light footer were fighting the theme.!importantoverrides and the--gb-*bridge. The remaining!importants are on the hero, where Bootstrap's own!importantutilities still appear in markup.Small fix
In the retail customers list, Delete is now
btn-smlike View and Edit.btn-group-smdoesn't reach a button nested in a<form>.Tests
GbaThemeResourcesTest(DC):GbaLayoutPagesTest(TP; the first Third Party test that boots a full Spring context):/,/login,/TermsOfServiceand/UsagePolicyrender anonymously with the GBA layout, SRI assets and no root-relative/aboutlink;Test plan
mvn verifyDC: 178 tests pass (+6); TP: 51 pass (+7, 2 skipped, pre-existing)localprofile, logged in as the dev admin: dashboard, Change Password (footer pinned to the bottom, full width,parentElement=BODY) and Retail Customers (white table, green header row)🤖 Generated with Claude Code