-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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: v5 hover:swap-active produces unwanted behavior #3375
Comments
Thank you @amir20
for reporting issues. It helps daisyUI a lot 💚
|
Just to confirm, this is not fixed yet? |
I'm working on it but I won't promise, as we don't have control over how The selector is something like .swap-rotate {
&.swap-active .swap-on {
rotate: 0deg;
}
} Tailwind won't generate But in general, may I ask if you want to change an icon on hover, why use swap? You can use Tailwind's
|
Thanks for the response! You are quick given all the things going on.
I wouldn't say that's obvious because then why have
I thought Tailwind support nested selectors natively. I'll have to look deeper when I can. |
Oh I guess if I do use the |
Try something like this for now: In general Tailwind CSS 4 simplified a lot of selector behaviors, which is a good thing. But not all behaviors are going to be 1:1 to the previous version. Also I will let you know. |
Thanks. I wouldn't spend a whole amount time on this only. It's probably NOT worth the time. I was hoping somehow to reuse as much as possible from DaisyUI instead of rolling my own. But if it comes to that then I could do it this way. |
What version of daisyUI are you using?
v5-beta
Which browsers are you seeing the problem on?
No response
Reproduction URL
https://play.tailwindcss.com/0UJ1LRDSK1
Describe your issue
@saadeghi not sure if you are aware of these two bugs.
swap swap-rotate hover:swap-active
andswap swap-flip hover:swap-active
do not work as expected.The text was updated successfully, but these errors were encountered: