Skip to content

Commit

Permalink
Fix from reviews
Browse files Browse the repository at this point in the history
  • Loading branch information
louismaximepiton committed Aug 22, 2022
1 parent b3b2573 commit 1e9d68b
Show file tree
Hide file tree
Showing 3 changed files with 225 additions and 48 deletions.
5 changes: 3 additions & 2 deletions scss/_side-navigation.scss
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
--#{$prefix}offcanvas-padding-y: 0;
--#{$prefix}offcanvas-border-width: 0;

flex-shrink: 0;
width: var(--#{$prefix}side-nav-width);
height: 100%;
overflow: hidden auto;
Expand Down Expand Up @@ -50,6 +51,7 @@
--#{$prefix}accordion-body-padding-x: 0;
--#{$prefix}accordion-body-padding-y: 0;
--#{$prefix}accordion-btn-font-size: 1rem;
--#{$prefix}accordion-btn-font-weight: #{$font-weight-normal};

.accordion-body {
--#{$prefix}side-nav-item-padding-left: calc(var(--#{$prefix}side-nav-icon-size) + var(--#{$prefix}side-nav-icon-margin-x) * 2); // stylelint-disable-line function-disallowed-list
Expand Down Expand Up @@ -84,13 +86,12 @@
display: flex;
align-items: center;
width: 100%;
height: var(--#{$prefix}side-nav-item-height);
min-height: var(--#{$prefix}side-nav-item-height);
padding: 0 0 0 var(--#{$prefix}side-nav-item-padding-left);
margin: 0;
overflow: hidden;
color: var(--#{$prefix}side-nav-color);
text-decoration: none;
white-space: nowrap;
background-color: var(--#{$prefix}side-nav-bg);
border: 0;

Expand Down
6 changes: 6 additions & 0 deletions site/assets/scss/_component-examples.scss
Original file line number Diff line number Diff line change
Expand Up @@ -242,6 +242,12 @@
}

// Boosted mod
.side-nav-example {
@include media-breakpoint-up(lg) {
height: 21.875rem;
}
}

.simple-list-example-scrollspy .active {
color: var(--#{$prefix}link-hover-color);
}
Expand Down
Loading

0 comments on commit 1e9d68b

Please sign in to comment.