diff --git a/CODE-AUDIT.md b/CODE-AUDIT.md index 8352726..7f06c66 100644 --- a/CODE-AUDIT.md +++ b/CODE-AUDIT.md @@ -607,7 +607,7 @@ token system + shared primitives (UI/SIMP — high value, larger effort) · i18n - [ ] **L117 — Terminal/error-log "empty" aren't centered blocks** like the other empty states (inline text instead). - [ ] **L118 — Cross-component focus via hardcoded id.** App focuses the URL field with `document.getElementById('aerofetch-url')` — brittle coupling into DownloadBar's markup. -- [ ] **L119 — "No items" copy varies** — "No … yet" (most) vs "Output will appear here." (Terminal) vs "No errors logged." (Diagnostics). +- [x] **L119 — "No items" copy varies** — "No … yet" (most) vs "Output will appear here." (Terminal) vs "No errors logged." (Diagnostics). - [ ] **L120 — Fluent `Card` vs hand-styled `div` cards.** Only Settings/Onboarding use ``; every other card surface is a bespoke styled `
`. - [ ] **L121 — Raw scrim literal.** CommandPalette backdrop is `rgba(0,0,0,0.32)` (the only raw rgba in @@ -677,11 +677,11 @@ token system + shared primitives (UI/SIMP — high value, larger effort) · i18n in the tray" is a three-sentence paragraph with a parenthetical. Normalize hint length/voice. - [x] **L150 — "PO token" capitalization varies** — `PO Token` (ipc.ts/ROADMAP) vs `PO token` / `Proof-of-Origin token` (SettingsView). Pick one. -- [ ] **L151 — Mixed range dashes.** En-dash in prose ("2–3 is a good balance") vs hyphen in time-range +- [x] **L151 — Mixed range dashes.** En-dash in prose ("2–3 is a good balance") vs hyphen in time-range placeholders ("1:30-2:00"). Choose one convention. - [ ] **L152 — Search-field sizing differs per screen.** History `minWidth 180 / maxWidth 320`, Settings inline `width: 100%`, DownloadBar `flexGrow`. No shared search-input width. -- [ ] **L153 — Generic "Dismiss" aria-labels.** Three close buttons (DownloadBar suggestion + dup, +- [x] **L153 — Generic "Dismiss" aria-labels.** Three close buttons (DownloadBar suggestion + dup, Library suggestion) all read just "Dismiss" — ambiguous to a screen reader. Name what's dismissed. - [x] **L154 — Instructional text in an aria-label.** Sidebar theme-cycle button is `aria-label="Theme: Dark. Click to change."` — "Click to change" is UI instruction, not a name. diff --git a/src/renderer/src/components/DownloadBar.tsx b/src/renderer/src/components/DownloadBar.tsx index 5a0e0be..c77b725 100644 --- a/src/renderer/src/components/DownloadBar.tsx +++ b/src/renderer/src/components/DownloadBar.tsx @@ -588,7 +588,7 @@ export function DownloadBar(): React.JSX.Element { appearance="subtle" icon={} onClick={dismissSuggestion} - aria-label="Dismiss" + aria-label="Dismiss suggested link" />
)} @@ -605,7 +605,7 @@ export function DownloadBar(): React.JSX.Element { appearance="subtle" icon={} onClick={() => setDup(null)} - aria-label="Dismiss" + aria-label="Dismiss duplicate warning" /> )} diff --git a/src/renderer/src/components/LibraryView.tsx b/src/renderer/src/components/LibraryView.tsx index 1cebc47..d7fd4fe 100644 --- a/src/renderer/src/components/LibraryView.tsx +++ b/src/renderer/src/components/LibraryView.tsx @@ -541,7 +541,7 @@ export function LibraryView(): React.JSX.Element { appearance="subtle" icon={} onClick={clip.dismiss} - aria-label="Dismiss" + aria-label="Dismiss suggested link" /> )} diff --git a/src/renderer/src/components/SettingsView.tsx b/src/renderer/src/components/SettingsView.tsx index 45649ab..b361226 100644 --- a/src/renderer/src/components/SettingsView.tsx +++ b/src/renderer/src/components/SettingsView.tsx @@ -536,7 +536,7 @@ export function SettingsView(): React.JSX.Element { {errorEntries.length === 0 ? ( - No errors logged. + No errors yet. ) : (
{errorEntries.slice(0, 20).map((e) => ( diff --git a/src/renderer/src/components/TerminalView.tsx b/src/renderer/src/components/TerminalView.tsx index ae275de..f5163cd 100644 --- a/src/renderer/src/components/TerminalView.tsx +++ b/src/renderer/src/components/TerminalView.tsx @@ -205,7 +205,7 @@ export function TerminalView(): React.JSX.Element {
         {lines.length === 0 ? (
-          Output will appear here.
+          No output yet.
         ) : (
           lines.map((l) => (