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: Buttons with state colors do not have animation on hover #1790

Closed
kyngs opened this issue May 2, 2023 · 4 comments
Closed

bug: Buttons with state colors do not have animation on hover #1790

kyngs opened this issue May 2, 2023 · 4 comments
Assignees

Comments

@kyngs
Copy link

kyngs commented May 2, 2023

What version of daisyUI are you using?

2.51.6

Describe your issue

There's no animation when hovering over buttons such as info, success, warning, or error. This issue is reproducible on the official site:

simplescreenrecorder-2023-05-02_13.21.54.mp4

What browsers are you seeing the problem on?

Chrome, Firefox

Reproduction URL (optional)

No response

@saadeghi saadeghi self-assigned this May 2, 2023
@dtcooper
Copy link

+1 on this one! Thanks :)

@kyngs
Copy link
Author

kyngs commented Jun 29, 2023

+1 on this one! Thanks :)

Unfortunately, it kinda seems like this got forgotten.

@saadeghi
Copy link
Owner

saadeghi commented Jun 30, 2023

It's not forgotten, but this improvement is expensive in file size and I don't think it's worth it.
Currently there are primary-focuse, secondary-focus, etc colors but there's no error-focus, info-focus, warning-focus, success-focus colors.
On the other hand there's a need for a soft version of these colors (because an alert with soft blue background and normal blue text looks better than normal blue background and white text. But for a button, blue background and white text looks better)
I haven't yet find a way to handle all these colors without increasing the amount of color names and making it more confusing to make themes. And keeping the number of CSS variables low.

I was planning to use CSS color-mix function because it solves all these issues but the browser support is not good yet.

@saadeghi
Copy link
Owner

Fixed in daisyUI 4

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

No branches or pull requests

3 participants