-
Notifications
You must be signed in to change notification settings - Fork 2.7k
ui feedback on assistant refresh #5269
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
ui feedback on assistant refresh #5269
Conversation
✅ Deploy Preview for continuedev canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was bothering me a bit too, thanks for the improvement! And videos as always 🙏
Just one nitpick, I prefer our spin-slow
since the default animate-spin
in Tailwind feels too fast for me in most cases.
buttonRef.current?.click(); | ||
}} | ||
> | ||
<ArrowPathIcon className="text-lightgray h-2.5 w-2.5" /> | ||
<ArrowPathIcon className={cn("text-lightgray h-2.5 w-2.5", loading && 'animate-spin')} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<ArrowPathIcon className={cn("text-lightgray h-2.5 w-2.5", loading && 'animate-spin')} /> | |
<ArrowPathIcon className={cn("text-lightgray h-2.5 w-2.5", loading && 'spin-slow')} /> |
https://github.com/continuedev/continue/blob/main/gui/tailwind.config.cjs#L26
my local assistans in ~/.continue/assistants/*.yaml are not loaded anymore?! |
Description
Added loading in reload icon present inside assistant selection menu. This adds a user UI feedback indicating that the config is being reloaded instead of waiting for "Config refreshed" message which appears after delay.
Checklist
Screenshots
before.mp4
after.mp4
Testing instructions