Skip to content

Input's default loading icon needs improvement #4162

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

Closed
alectrocute opened this issue May 15, 2025 · 8 comments
Closed

Input's default loading icon needs improvement #4162

alectrocute opened this issue May 15, 2025 · 8 comments
Labels
bug Something isn't working v3 #1289

Comments

@alectrocute
Copy link
Contributor

alectrocute commented May 15, 2025

Environment

N/A

Is this bug related to Nuxt or Vue?

Nuxt

Version

3.0.0

Reproduction

https://ui.nuxt.com/components/input#loading-icon

Description

Both icons, the one shown in the example i-lucide-repeat-2 and the component's default prop, i-lucide-refresh-cw look bad when rotated. I think there's a lot of room for improvement, these icons aren't meant to be animated.

Luckily, there are some Lucide icons which are designed to be animated.

How about https://lucide.dev/icons/loader-circle?

Or https://lucide.dev/icons/loader?

Additional context

No response

Logs

@alectrocute alectrocute added bug Something isn't working v3 #1289 triage labels May 15, 2025
@huongphamx
Copy link

I think It's personal preference though. You can always set your own default loading token in app.config.ts file.

@huongphamx
Copy link

and so, I think this is not a bug

@alectrocute
Copy link
Contributor Author

and so, I think this is not a bug

We have an icon spinning that’s not meant to be spun. That’s a bug.

My personal preference is good UI.

@huongphamx
Copy link

well, so you can build your own UI lib to match your high "preference"

@alectrocute
Copy link
Contributor Author

well, so you can build your own UI lib to match your high "preference"

Your replies are off-topic and unwelcome, you aren’t helping the project, you’re just trying to start arguments.

I’m not going to make my own UI lib. That’s such an obtuse thing to suggest. I made a PR hours ago to fix this issue, what are you doing?

@huongphamx
Copy link

You are helping by bring your preference and force busy dev team to change to what you want, got that.

It's just your preference, and you alway can 'build your own UI' by config in app.config.ts file.

But well, if you see that as unwelcome, feel free to ignore me

@alectrocute
Copy link
Contributor Author

You are helping by bring your preference and force busy dev team to change to what you want, got that.

It's just your preference, and you alway can 'build your own UI' by config in app.config.ts file.

I already did the work, how is that forcing the busy dev team to do anything?

But well, if you see that as unwelcome, feel free to ignore me

Yes please!

@alectrocute
Copy link
Contributor Author

@huongphamx #4163

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

No branches or pull requests

3 participants