-
Notifications
You must be signed in to change notification settings - Fork 54
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: remove space that is messing with button alignment. #1341
Conversation
Test Version for this PR was deployed Built with commit 7f5ece1 |
Looks like the snapshots images need to be updated, but I cannot find any docs on this. Also need to update the docs site css. According to https://github.com/SAP/fundamental/wiki/Contribution-Guidelines#5-run-the-documentation, I should just have to |
docs/css/fiori-fundamentals.css
Outdated
@@ -3549,7 +3549,7 @@ This will render 4 boxes spanning 2 cols each indented 2 cols | |||
-webkit-box-shadow: 0 0 0 1px var(--fd-color-action-focus); | |||
box-shadow: 0 0 0 1px var(--fd-color-action-focus); } | |||
[dir="rtl"] .fd-alert__close, .fd-alert__close[dir="rtl"] { | |||
left: 4px; | |||
left: 0; |
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 think this PR should be held until #1345 is merged. The files fui-site.css
and fiori-fundamentals.css
are no longer required to be updated in source control (since they are generated).
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.
#1345 is merged :)
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.
Looks good. 🚢
fixes #1271
orig:
fixed: