dandri/meshcore-analyzer
1
0
Fork 0
mirror of https://github.com/Kpa-clawbot/meshcore-analyzer.git synced 2026-05-14 07:55:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(#1128): define --text-primary, --bg-hover, --primary aliases (lint green)

Browse Source 
scripts/check-css-vars.js (added in previous commit) flagged three
var(--name) references in public/style.css that resolved to
nothing because the names were never defined:

  style.css:2628  var(--text-primary)   .rf-detail-meta
  style.css:2675  var(--bg-hover)       .rf-detail-close:hover
  style.css:2924  var(--primary)        .tools-card:hover

Same root cause as Bug 4 (--surface) in PR #1131. Alias them to
existing tokens in :root so they resolve in both light and dark
themes (the dark blocks redefine --text / --hover-bg / --accent
upstream of the aliases, so cascade flows naturally).

After this commit: `node scripts/check-css-vars.js` exits 0,
869 var() refs, 64 definitions, 0 undefined.

Refs #1128
This commit is contained in:
Kpa-clawbot
2026-05-06 06:50:12 +00:00
parent 0d4660fad5
commit 1369d1e346
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
public/style.css
+7
View File
@@ -117,6 +117,13 @@
--content-bg: var(--surface-0);
--card-bg: var(--surface-1);
--hover-bg: rgba(0,0,0, 0.04);
/* #1128 followup: caught by scripts/check-css-vars.js — these were
* referenced without a fallback in .rf-detail-meta, .rf-detail-close
* and .tools-card and would resolve transparent / inherit. Alias them
* to existing tokens. */
--text-primary: var(--text);
--bg-hover: var(--hover-bg);
--primary: var(--accent);
--trace-ghost-color: #94a3b8;
/* #1128: documented z-index scale. Use these custom props for any new