Skip to content
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

Closed
stefwalter opened this issue Jul 21, 2014 · 7 comments
Closed

Throbber in buttons #898

stefwalter opened this issue Jul 21, 2014 · 7 comments
Assignees

Comments

@stefwalter
Copy link
Contributor

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?

@stefwalter stefwalter added this to the Fedora 21 milestone Jul 21, 2014
@andreasn
Copy link
Contributor

I see this too.
The waiting class has a set hight and width of 28px that is bigger than the button.
I'm pretty sure I've seen throbber in buttons in other places of the UI as well.

@stefwalter
Copy link
Contributor Author

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?

@stefwalter
Copy link
Contributor Author

@porscheyin noted that this causes dialogs to resize and flicker. We need advice from @andreasn here.

@mvollmer mvollmer removed this from the Fedora 21 Beta milestone Sep 16, 2014
@andreasn
Copy link
Contributor

Lets try and move away from all throbbers in buttons then. In how many places do we do that currently?

@andreasn
Copy link
Contributor

So the only place I could find it is the Add Server dialog. Anywhere else?

@andreasn andreasn assigned porscheyin and unassigned andreasn Sep 24, 2014
@dperpeet
Copy link
Contributor

dperpeet commented Dec 1, 2015

@andreasn I think we have successfully moved away from this. Can you confirm?

@dperpeet dperpeet assigned andreasn and unassigned porscheyin Dec 1, 2015
@stefwalter stefwalter added needsdesign and removed question Further information is requested labels Mar 21, 2016
@martinpitt
Copy link
Member

This hasn't been an issue in years, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants