Skip to content

refactor(ui): centralize destructive ghost button colors in EMCN - #7988

Open
BillLeoutsakosvl346 wants to merge 5 commits into
codex/control-fix-modal-fieldsfrom
codex/control-fix-destructive-ghost-buttons
Open

BillLeoutsakosvl346 wants to merge 5 commits into
codex/control-fix-modal-fieldsfrom
codex/control-fix-destructive-ghost-buttons

Conversation

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor

Summary

Eleven product delete/remove buttons override EMCN Button colors locally. Add ghost-destructive (red at rest and on hover) and ghost-destructive-muted (muted at rest, red on hover) to the existing Button, and migrate all nine always-red and two muted consumers. Both use existing global color tokens.

The rendered class sets are identical. Geometry, icons, labels, handlers, disabled conditions, native button behavior and the input-format button's hover opacity remain unchanged. Existing Button variants and defaults are unchanged.

Product only: 12 source files, with landing excluded. All audit tooling and browser fixtures stay outside the branch; no dependencies or new tests.

Testing

  • 343 existing EMCN tests passed; EMCN/app type checks, repository lint, API validation and git diff --check passed.
  • Consumer AST comparison confirms only the variant and color classes changed at all 11 locations.
  • Real before/after Button and Trash renders have identical markup after class-order normalization. 110 Chromium comparisons match in light/dark idle, hover, focus, active and disabled states using compiled app CSS. All 210 existing variant/size/padding combinations render identical markup. These are isolated component checks, not end-to-end product navigation.
  • Final external category audit: 22 color overrides across 11 buttons → 0. Broader findings 1003 → 981, with unrelated findings unchanged. Full inventory still has unrelated findings and analysis gaps. Landing diff is empty.
  • Committed-diff audit: zero usage violations; four expected central-definition notices for the two new color variants.

@vercel

vercel Bot commented Sep 18, 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 23, 2026 2:25am 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 12 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 18, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the refactor preserves the migrated buttons’ rendered classes and interaction behavior.

Summary

The PR centralizes destructive ghost-button colors in EMCN and migrates the affected workflow and knowledge UI consumers. It also extracts the duplicated knowledge-tag summary markup into a shared component.

  • Adds always-red and muted-until-hover destructive ghost variants to the shared Button CVA.
  • Replaces consumer-local destructive color utilities while preserving geometry, handlers, disabled states, and accessible labels.
  • Reuses KnowledgeTagRow across the document-tag and knowledge-base tag modals.

Reviews (3) · Last reviewed commit: "refactor(ui): centralize destructive gho..."

@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the codex/control-fix-destructive-ghost-buttons branch from 76d44d6 to 16b81b5 Compare September 19, 2026 19:51
@BillLeoutsakosvl346
BillLeoutsakosvl346 changed the base branch from staging to codex/control-fix-modal-fields September 19, 2026 19:51
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptileai @cubic-dev-ai Please review the current head. The previous scored reviews predate the base update.

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

@greptileai @cubic-dev-ai Please review the current head. The previous scored reviews predate the base update.

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@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 12 files

Confidence score: 5/5

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

Re-trigger cubic

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

Before / after visual comparison

Destructive icon actions: shared Button treatments preserve the muted/red icon colors and geometry.

Within each numbered tile: first icon = before; second icon = after.

Rendered component fixtures with application CSS and local data, rather than authenticated product-page screenshots. These are representative visual examples, not end-to-end behavior coverage.

Current PR head checked at posting: 5653710ea0.

Before and after: buttons light

Additional theme/state comparisons

Before and after: buttons dark

@BillLeoutsakosvl346
BillLeoutsakosvl346 force-pushed the codex/control-fix-destructive-ghost-buttons branch from 2b16e72 to 5e8223b Compare September 22, 2026 20:47
@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@greptile

@BillLeoutsakosvl346

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete.

@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 13 files

Confidence score: 5/5

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

Re-trigger cubic

This branch was previously deployed

1 inactive deployment
Preview 54e1c011 Deployed Sep 23, 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