From eb8a0d30da0cc150e95d44bad998c13ee2defc25 Mon Sep 17 00:00:00 2001
From: Christian Cleberg <hello@cleberg.net>
Date: Mon, 20 Jul 2026 18:16:01 -0500
Subject: feat(a11y): semantic colour system (#21 phase 1)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Replaces every hard-coded colour with semantic asset colours that adapt to
light, dark, and Increase Contrast. Dark mode stays locked, so this is a
pure refactor: the audit reports the same findings before and after.

The accent is now blue rather than cyan, per the tech/DNS theme.

Why custom values rather than the system palette: every system colour
fails WCAG AA in light mode. Measured on white — systemYellow 1.51:1,
systemOrange 2.20:1, systemGreen 2.22:1, systemCyan 2.54:1, systemRed
3.55:1. All of them pass in dark mode, which is why the dark-locked app
looked fine, and why unlocking light mode was never a matter of deleting
.preferredColorScheme(.dark). Every new value clears 4.5:1 as text on its
page, its card, and its own 16% badge tint — the way AppStatusBadgeView
actually draws it.

The accent needed splitting in two. As text on a dark background it must
be light; as a fill behind a white label it must be dark. #4DA3FF reads
8.00:1 as text on black but 2.63:1 behind white text, so StatusInfo /
AccentColor cover the foreground role and AccentFill covers
.borderedProminent. AppOnAccent is the label colour for a solid fill and
flips by scheme.

Colours live in Shared/Colors.xcassets rather than the app catalog: the
Shared folder is already a synchronized group in all three targets, so
the widget and share extension pick the palette up with no project-file
surgery. AccentColor stays in the app catalog as the global tint — and is
now actually defined, having been an empty colorset that silently left
system controls rendering in stock blue while custom chrome used cyan.

Two deliberate visual changes: orange folds into StatusWarning and pink
into StatusCritical. They encoded the same severity as the colours they
now share, and both sites also carry a text label.

Audit findings drop 15 to 14, and one of the originals turned out to be
a phantom: the Inspect contrast failure was the Run button in its
disabled state, which WCAG 1.4.3 exempts. testInspectScreen now types a
domain first so the audit measures an enabled control. Findings also
carry the offending element now, so the remaining clipped-text items name
themselves ("No Portfolio Yet", "Search domains") instead of being
anonymous.
---
 DomainDigWidget/SweepLiveActivity.swift | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'DomainDigWidget/SweepLiveActivity.swift')

diff --git a/DomainDigWidget/SweepLiveActivity.swift b/DomainDigWidget/SweepLiveActivity.swift
index b99a070..b96f8bf 100644
--- a/DomainDigWidget/SweepLiveActivity.swift
+++ b/DomainDigWidget/SweepLiveActivity.swift
@@ -24,7 +24,7 @@ struct SweepLiveActivity: Widget {
                 DynamicIslandExpandedRegion(.bottom) {
                     VStack(alignment: .leading, spacing: 4) {
                         ProgressView(value: context.state.fractionComplete)
-                            .tint(.cyan)
+                            .tint(Color(.statusInfo))
                         if let current = context.state.currentDomain {
                             Text(current)
                                 .font(.caption2)
@@ -35,7 +35,7 @@ struct SweepLiveActivity: Widget {
                 }
             } compactLeading: {
                 Image(systemName: "arrow.trianglehead.2.clockwise")
-                    .foregroundStyle(.cyan)
+                    .foregroundStyle(Color(.statusInfo))
             } compactTrailing: {
                 Text("\(context.state.completed)/\(context.state.total)")
                     .font(.caption2)
@@ -43,7 +43,7 @@ struct SweepLiveActivity: Widget {
             } minimal: {
                 ProgressView(value: context.state.fractionComplete)
                     .progressViewStyle(.circular)
-                    .tint(.cyan)
+                    .tint(Color(.statusInfo))
             }
         }
     }
@@ -66,7 +66,7 @@ private struct SweepActivityBannerView: View {
             }
 
             ProgressView(value: context.state.fractionComplete)
-                .tint(.cyan)
+                .tint(Color(.statusInfo))
 
             HStack {
                 if let current = context.state.currentDomain {
@@ -83,12 +83,12 @@ private struct SweepActivityBannerView: View {
                 if context.state.changed > 0 {
                     Text("\(context.state.changed) changed")
                         .font(.caption2)
-                        .foregroundStyle(.orange)
+                        .foregroundStyle(Color(.statusWarning))
                 }
                 if context.state.warnings > 0 {
                     Text("\(context.state.warnings) warnings")
                         .font(.caption2)
-                        .foregroundStyle(.red)
+                        .foregroundStyle(Color(.statusCritical))
                 }
             }
         }
-- 
cgit v1.2.3