Skip to content

fix(ui): use defined status and interaction color tokens - #8012

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/control-fix-status-color-tokens
Sep 19, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
codex/control-fix-status-color-tokens

Conversation

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor

Several product status indicators referenced undefined CSS variables, leaving attention dots and progress fills transparent and warning/action text without its intended color.

Use the existing --brand-blue for progress and attention, --warning for the legacy-snapshot warning, and --selection for the download action and Cursor keyboard focus ring. Apply the unviewed-download color to its icon instead of overriding the Button. Layout, dimensions, and interactions remain unchanged.

Validation: all eight stale references removed by a tracked-source search; the external conformance audit removes all seven associated consumer findings with no new findings. Existing EMCN and browser-download tests, workspace type/lint checks, import resolution, and mandatory API validation pass. A browser fixture compiled from the real application CSS verifies both themes, matching dimensions, and keyboard focus colors; full live product flows were not exercised. Audit tools and visual-review files are kept outside this PR.

@vercel

vercel Bot commented Sep 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 19, 2026 5:57pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 6 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no actionable regression or repository-rule violation was identified.

Summary

This PR replaces stale, undefined color-variable references with defined semantic tokens while preserving layout and interaction behavior.

  • Uses --brand-blue for progress, completion, and attention indicators.
  • Uses --warning for the missing-snapshot warning.
  • Uses --selection for interactive action text and keyboard focus.
  • Limits the unviewed-download color to the Download icon rather than the surrounding Button.

Reviews (1) · Last reviewed commit: "fix(ui): use defined status and interact..."

@waleedlatif1
waleedlatif1 merged commit 14994dc into staging Sep 19, 2026
37 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/control-fix-status-color-tokens branch September 19, 2026 18:55

This branch was successfully deployed

1 active deployment
Preview 3fc392fc Deployed Sep 19, 2026 by vercel[bot]
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