This repository was archived by the owner on May 29, 2026. It is now read-only.
fix(brand): solid xxllnc tokens, fix typos, MDI app icons, remove dead assets#133
Merged
Merged
Conversation
…d assets Closes #118 — xxllnc primary/hover were translucent (#000000ad/#80); replaced with solid #000000/#333333 to comply with brand solid-colour rule and guarantee WCAG AA contrast on any background. Closes #119 — typo --nldesign-color-succes (missing 's') left success color undefined in the xxllnc set; corrected to --nldesign-color-success in both the nldesign and xxllnc palette sections. Same fix for alert-succes variants. Closes #120 — docs/static/img/logo.svg still used legacy cobalt #4376FC; replaced with canonical #21468B (Dutch flag blue, retired 2026-05-13). Closes #121 — all three app icon files contained an opacity="0.3" rect violating the solid-colours brand rule; replaced with a proper MDI palette-swatch glyph: fill="#fff" 24×24 for app.svg, no-fill for app-dark.svg, #21468B hexagon + white glyph for app-store.svg. Closes #124 — css/fonts/ was an unreferenced duplicate of css/systems/nldesign/fonts/; deleted all 8 font files (~2 MB of dead weight). Closes #125 — nederland-logo.svg and nederland-map.svg were byte-identical; only nederland-logo.svg is referenced (rijkshuisstijl.css); deleted the duplicate nederland-map.svg and the stale nederland-map-backup.svg.
This was referenced May 27, 2026
Closed
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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
#000000ad/#00000080); replaced with solid#000000/#333333to comply with brand solid-colour rule and guarantee WCAG AA contrast on any background. Comment added to flag upstream xxllnc maintainers.--nldesign-color-succes(missing 's') left success colour undefined in the xxllnc set. Corrected to--nldesign-color-successin both the nldesign and xxllnc palette sections. Same fix foralert-succesvariants.docs/static/img/logo.svgused legacy cobalt#4376FC; replaced with canonical#21468B(Dutch flag blue, retired 2026-05-13).opacity="0.3"rect violating solid-colours brand rule; replaced with proper MDI palette-swatch glyph:fill="#fff"24×24 forapp.svg, no-fill forapp-dark.svg,#21468Bhexagon + white glyph forapp-store.svg.css/fonts/was an unreferenced duplicate ofcss/systems/nldesign/fonts/; deleted all 8 font files (~2 MB dead weight). No stylesheet references this directory.nederland-logo.svgandnederland-map.svgwere byte-identical (confirmed md5); onlynederland-logo.svgis referenced (rijkshuisstijl.cssline 89); deleted duplicatenederland-map.svgand stalenederland-map-backup.svg.Test plan
#21468Bcobalt (not the old#4376FC)css/systems/nldesign/fonts/)nederland-logo.svgwhich is kept)