Commits

Tim Donohue authored and GitHub committed 7b42446c115 Merge
Merge pull request #1861 from the-library-code/PRs/FixedHeader768px

Workaround missing navbar @768px for dspace and custom templates
No tags
gidlmaster

src/app/navbar/navbar.component.scss

Modified
1 1 nav.navbar {
2 2 border-bottom: 1px var(--bs-gray-400) solid;
3 3 align-items: baseline;
4 4 }
5 5
6 6 /** Mobile menu styling **/
7 -@media screen and (max-width: map-get($grid-breakpoints, md)) {
7 +@media screen and (max-width: map-get($grid-breakpoints, md)-0.02) {
8 8 .navbar {
9 9 width: 100vw;
10 10 background-color: var(--bs-white);
11 11 position: absolute;
12 12 overflow: hidden;
13 13 height: 0;
14 14 &.open {
15 15 height: 100vh; //doesn't matter because wrapper is sticky
16 16 }
17 17 }
18 18 }
19 19
20 20 @media screen and (min-width: map-get($grid-breakpoints, md)) {
21 21 .reset-padding-md {
22 22 margin-left: calc(var(--bs-spacer) / -2);
23 23 margin-right: calc(var(--bs-spacer) / -2);
24 24 }
25 25 }
26 26
27 27 /* TODO remove when https://github.com/twbs/bootstrap/issues/24726 is fixed */
28 28 .navbar-expand-md.navbar-container {
29 - @media screen and (max-width: map-get($grid-breakpoints, md)) {
29 + @media screen and (max-width: map-get($grid-breakpoints, md)-0.02) {
30 30 > .container {
31 31 padding: 0 var(--bs-spacer);
32 32 }
33 33 padding: 0;
34 34 }
35 35 }
36 36
37 37 .navbar-nav {
38 38 ::ng-deep a.nav-link {
39 39 color: var(--ds-navbar-link-color);

Everything looks good. We'll let you know here if there's anything you should know about.

Add shortcut