Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bug]: css var destructive-foreground is same color as destructive #6832

Open
2 tasks done
KyGuy2002 opened this issue Mar 2, 2025 · 2 comments · May be fixed by #6846
Open
2 tasks done

[bug]: css var destructive-foreground is same color as destructive #6832

KyGuy2002 opened this issue Mar 2, 2025 · 2 comments · May be fixed by #6846
Labels
bug Something isn't working

Comments

@KyGuy2002
Copy link

KyGuy2002 commented Mar 2, 2025

Describe the bug

In a recent commit #6693 the destructive-foreground color was changed to red, which is the same color as destructive. This makes it impossible to see the text in destructive buttons or other elements using those variables.

Image

Affected component/components

Button

How to reproduce

Add a button element to the page and notice how the text is not visible at all. This only applies in light mode.

Codesandbox/StackBlitz link

No response

Logs

System Info

N/A

Before submitting

  • I've made research efforts and searched the documentation
  • I've searched for existing issues
@KyGuy2002 KyGuy2002 added the bug Something isn't working label Mar 2, 2025
@MrADOY
Copy link

MrADOY commented Mar 3, 2025

Hello

Same here ? 😔

@Jacksonmills
Copy link
Contributor

I'll make a PR for this fix

Jacksonmills added a commit to Jacksonmills/ui that referenced this issue Mar 4, 2025
…erlap

Changes the `destructive-foreground` CSS variable to the correct color to prevent it from matching the `destructive` color, ensuring better visual differentiation.

Fixes shadcn-ui#6832
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants