refactor(ui): centralize destructive ghost button colors in EMCN - #7988
BillLeoutsakosvl346 wants to merge 5 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
76d44d6 to
16b81b5
Compare
|
@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. |
Before / after visual comparisonDestructive 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: |
4360284 to
cfa8a5c
Compare
2b16e72 to
5e8223b
Compare
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |


Summary
Eleven product delete/remove buttons override EMCN Button colors locally. Add
ghost-destructive(red at rest and on hover) andghost-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
git diff --checkpassed.