-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Add margin variable for breadcrumb #23138
Conversation
scss/_variables.scss
Outdated
@@ -787,6 +787,8 @@ $breadcrumb-padding-y: .75rem !default; | |||
$breadcrumb-padding-x: 1rem !default; | |||
$breadcrumb-item-padding: .5rem !default; | |||
|
|||
$breadcrumb-margin: 0 0 1rem! default; |
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.
Invalid CSS after "... 0 0 1rem!": expected identifier, was " default;"
@@ -1,6 +1,6 @@ | |||
.breadcrumb { | |||
padding: $breadcrumb-padding-y $breadcrumb-padding-x; | |||
margin-bottom: 1rem; |
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.
Leave it as margin-bottom
, please. There's no need to reset a non-existant margin-top
, -right
, and -left
if we have no intention of ever applying those values (which I don't :p).
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.
Ok then i'll keep margin-bottom
only.
Will do the change.
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.
@mdo Changes done.
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.
@mdo Can you please review and close this PR if we don't have any more changes?
This'll go in for Beta 2 at the earliest. |
Ok Thanks @mdo |
I have added a variable for breadcrumb margin to
_variables.scss
and replaced static margin in_reboot.css
with that variable.Fixes #23016.