Skip to content

chore(brand): align app-store hex to canonical cobalt #21468B#762

Merged
rubenvdlinde merged 2 commits into
developmentfrom
chore/brand-cobalt-21468b
May 21, 2026
Merged

chore(brand): align app-store hex to canonical cobalt #21468B#762
rubenvdlinde merged 2 commits into
developmentfrom
chore/brand-cobalt-21468b

Conversation

@rubenvdlinde
Copy link
Copy Markdown
Contributor

Single-file change to img/app-store.svg: hex polygon fill updated from legacy #4376FC to canonical #21468B (--c-blue-cobalt). Part of the fleet sweep aligning the app icon to the brand cobalt — see design-system colors.html for the rationale. No code or behavior changes.

The icon hex was filled with the legacy '#4376FC' (the
conduction-design-tokens 'Conduction Cobalt'). The design-system
has retired this in preview/colors.html with the caption

> Our legacy blue (#4376FC) read as 'cloud / SaaS', light, frisky,
> lacking weight. Cobalt is the official blue of the Dutch flag,
> Dutch by birthright, dark enough to feel serious, light enough
> to stay blue, and AAA-contrast on white so it can carry body
> text.

Swap to #21468B (--c-blue-cobalt in tokens.css) so the app-store
hex matches the navbar wordmark and the rest of the brand chrome.
Fleet sweep 2026-05-13.
@github-actions
Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/openconnector @ 0b6e473

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
composer ✅ 148/148
npm ❌ 1/573 denied
PHPUnit ⏭️
Newman ⏭️
Playwright ⏭️

❌ Denied npm licenses

Package Version License
@fortawesome/free-solid-svg-icons 6.7.2 (CC-BY-4.0 AND MIT)

Quality workflow — 2026-05-13 09:25 UTC

Download the full PDF report from the workflow artifacts.

Copy link
Copy Markdown
Contributor

@WilcoLouwerse WilcoLouwerse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found — clean PR ✅

@github-actions
Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/openconnector @ 9cdf27b

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
composer ✅ 148/148
npm ✅ 674/674
PHPUnit ⏭️
Newman ⏭️
Playwright ⏭️

Quality workflow — 2026-05-19 04:03 UTC

Download the full PDF report from the workflow artifacts.

@rubenvdlinde rubenvdlinde merged commit 4e1743a into development May 21, 2026
22 checks passed
@rubenvdlinde rubenvdlinde deleted the chore/brand-cobalt-21468b branch May 21, 2026 19:33
rubenvdlinde added a commit that referenced this pull request May 22, 2026
Resolves 30-file conflict between i18n's Tier-4 refactor (OR-adoption +
PHPCS docblock harmonisation + manifest v2 schema URL flip) and the 9
commits dev accumulated independently (#823 LogIndex wrapper, #842
.php-cs-fixer cleanup, #849 root-config sync + phpmd cleanup, #727
cross-entity slug refs, #752 PDOK adapter, #762 brand cobalt, #767
specter spec, #703 .gitignore harmonise, #679 openspec sync workflows).

Resolution strategy:
- 17 DU conflicts (Db classes + ExportService) — confirmed i18n's
  deletions (Tier-4 OR-adoption: data moved off bespoke Db/ classes
  to OR-backed objects).
- l10n/en.json + l10n/nl.json — took HEAD's union (translation work
  was done on i18n).
- composer.lock — took HEAD's (i18n had it regenerated for new deps).
- src/manifest.json — took HEAD (v2 schema URL + 2-space indent + the
  typed-primitive page shapes; whitespace-only conflict otherwise).
- 8 UU conflicts on PHP controllers/services + routes.php + registry.js
  — took HEAD (i18n). The systematic pattern: i18n calls the new OR
  API (->getObject()) while dev still references the now-deleted Db
  classes (->jsonSerialize()). Dev's references would break at
  runtime against i18n's structural state; HEAD is the only
  internally-consistent resolution.

All conflict-resolved files: 0 markers remaining, PHP syntactically
valid. Manifest still validates clean against v2 schema 2.7.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants