bug: prefers-reduced-motion shouldn't stop Loading component animation

[marklbarlow]

Reproduction URL (Required)

https://daisyui.com/components/loading/

What version of daisyUI are you using?

5.6.0+

Which browsers are you seeing the problem on?

All browsers

Describe your issue

#4299 was recently merged and released as part of v5.6.

With reduced motion selected this now completely disables animation and make the app look like it is non-responsive. This exact scenario was commented on by @saadeghi in the PR but it still seems to have been merged without further changes.

I believe this PR should be reverted. Whereas menu flyouts etc are eye-candy, the whole purpose of a loading animation is to show that something is happening. Without the animation it looks like the app is unresponsive.

If a user wants to honour this, they should handle the preference in their own code and provide an alternative "Loading..." text etc to display instead.

If it's considered essential to keep this as part of Daisy, the framework should provide this feature and / or show the loading indicator with reduced motion, rather than no motion at all. As it stands, the component is now unusable.

[github-actions]

Thank you @marklbarlow for reporting issues. It helps daisyUI a lot 💚
I'll be working on issues one by one. I will help with this one as soon as a I find a solution.
In the meantime providing more details and reproduction links would be helpful.

[saadeghi]

I agree. I think a slower animation will be better than disabling the animation.

It will be fixed by tomorrow 👍

[marklbarlow]

Thanks a lot for the quick response. Much appreciated. And many thanks for all your hard work on this fantastic library! 👍

[saadeghi]

Fixed in 5.6.7. Thanks for reporting 💚