RomanNum3ral
/
netbox
mirror of https://github.com/netbox-community/netbox.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #23020 from netbox-community/23007-sidenav-javascript-uses-a-different-breakpoint-from-the 

Closes #23007: Align sidebar initialization with the responsive layout breakpoint
This commit is contained in:
bctiemann 2026-08-24 17:51:44 -04:00 committed by GitHub
parent b08799860f 105e58a433
commit b7e02fe098
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
netbox/project-static/dist/netbox.js vendored
View File

File diff suppressed because one or more lines are too long

6
netbox/project-static/dist/netbox.js.map vendored
View File

File diff suppressed because one or more lines are too long

5
netbox/project-static/src/sidenav.ts
View File

@ -4,6 +4,9 @@ import { getElements, isElement } from './util';
type NavState = { pinned: boolean };
type BodyAttr = 'show' | 'hide' | 'hidden' | 'pinned';
// Keep in sync with Bootstrap's `lg` breakpoint and `navbar-expand-lg` in base/layout.html.
const SIDENAV_DESKTOP_MEDIA = '(min-width: 992px)';
class SideNav {
/**
* Sidenav container element.
@ -63,7 +66,7 @@ class SideNav {
toggler.addEventListener('click', event => this.onMobileToggle(event));
}
if (window.innerWidth >= 1200) {
if (window.matchMedia(SIDENAV_DESKTOP_MEDIA).matches) {
if (this.state.get('pinned')) {
this.pin();
} else {