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 missing button-block-appender icon on Firefox 71 and Safari 13 #19366
fix missing button-block-appender icon on Firefox 71 and Safari 13 #19366
Conversation
outline: $border-width dashed $dark-gray-150; | ||
width: 100%; | ||
height: 100%; |
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.
I'm not sure about this rule as it's dependent on the parent element and this is a generic component, should this be auto
instead of100%
?
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.
Yes, I agree! :)
I actually realised this after I opened the PR, but forgot to commit the update.
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.
Thanks for the fix
Description
Fixes #19246
How has this been tested?
Manual testing using:
Screenshots
Before:
After:
Types of changes
Small style updates.
Checklist: