Skip to content

Commit 8cbcf08

Browse files
committed
feat: tweak the internal spacing for the Navbar so extra elements nested inside the main title section will automatically get spacing in between each item + eliminate the need for items themselves adding their own space
1 parent ab577a8 commit 8cbcf08

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

packages/components/bolt-navbar/src/navbar.scss

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,10 @@ bolt-navbar {
4343
@include bolt-padding-left(small);
4444
@include bolt-padding-right(small);
4545

46+
> *:not(:last-child){
47+
@include bolt-margin-right(small);
48+
}
49+
4650
@include bolt-mq($bolt-navbar-xsmall-bp){
4751
@include bolt-padding-left(medium);
4852
@include bolt-padding-right(medium);
@@ -118,9 +122,9 @@ bolt-navbar {
118122
}
119123
}
120124

121-
// Don't include these inner "full-bleed" styles till Navbar is updated to support theming classes internally,
125+
// Don't include these inner "full-bleed" styles till Navbar is updated to support theming classes internally,
122126
// otherwise full bleed + center styling combos are broken
123-
// @todo: uncomment once the [width*="full"] updates from above ^ are addressed
127+
// @todo: uncomment once the [width*="full"] updates from above ^ are addressed
124128
// &--full {
125129
// @include bolt-full-bleed;
126130
// }
@@ -148,13 +152,14 @@ bolt-navbar {
148152
color: bolt-theme(headline);
149153
white-space: nowrap;
150154

155+
> *:not(:last-child){
156+
@include bolt-margin-right(small);
157+
}
158+
151159
@supports (--css: variables) {
152160
padding-bottom: var(--bolt-vspacing, $bolt-navbar-vspacing-small);
153161
}
154162

155-
@include bolt-mq(medium){
156-
@include bolt-margin-right(medium); // Maintain space between items
157-
}
158163

159164
&:only-child {
160165
@include bolt-margin-right(auto);

0 commit comments

Comments
 (0)