-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Buttons] Make sure button have sufficient click target #10128
Comments
Looking at the most common |
@heyjoethomas I'm wondering if this feedback was initial change aversion to the buttons getting smaller. I haven't heard it being an issue recently. |
That is my best guess. The buttons did get a bit smaller for uplift. Basic buttons went from 36px tall -> 28px tall. So my guess is that just threw people off. |
Want to make sure before i close: is there any action to be taken to address this issue? |
I don't think so no. The buttons were obviously designed intentionally and
the spacing between buttons around the admin is consistent. I think this is
just reaction from the design change.
|
We've received some merchant feedback that the smaller Uplift buttons are harder to click. I think this has less to do with the size and more to do about the space around it, especially Button Group.
Logging this so a designer can have a lot to see if there's any places where Buttons don't have sufficient padding.
The text was updated successfully, but these errors were encountered: