-
Notifications
You must be signed in to change notification settings - Fork 157
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
Fix spaceing between buttons #3967
Conversation
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.
Using !important
in lots of places can make things hard to customize in the future. Are we sure we need these everywhere? I wonder if we could achieve it with regular CSS specificity logic.
Some docs on the subject: https://developer.mozilla.org/en-US/docs/Web/CSS/Specificity
Love the thorough write up with pics 👍
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.
Looking good! 👍
Closes #3891
What changed?
Set a fixed space between buttons to be
12px
before:
after:
before:
after:
before:
after:
before:
after: