-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Throbber in buttons #898
Comments
I see this too. |
The reason the waiting class has that size is so it can be used next to buttons. I guess we need a different class if it's used inside of a button. But do we really want the throbber to be inside a button? |
@porscheyin noted that this causes dialogs to resize and flicker. We need advice from @andreasn here. |
Lets try and move away from all throbbers in buttons then. In how many places do we do that currently? |
So the only place I could find it is the Add Server dialog. Anywhere else? |
@andreasn I think we have successfully moved away from this. Can you confirm? |
This hasn't been an issue in years, closing. |
We have throbbers that show up in buttons, like on the 'Add Server' dialog. It changes the size of the buttons and causes flicker. Should we just disable the button and put the throbber in the empty space to the far left?
The text was updated successfully, but these errors were encountered: