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

fix(ui): Simplify sidebar initialization logic 

Replace redundant conditional blocks with single if-else statement and
move resize listener outside viewport check. Changes width threshold
from `>` to `>=` for consistency with standard breakpoint behavior.

Fixes #22930
This commit is contained in:
Martin Hauser 2026-08-20 15:03:46 +02:00 committed by Jeremy Stretch
parent 58ac88bc51
commit b6cc7d811f
3 changed files with 10 additions and 15 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

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

File diff suppressed because one or more lines are too long

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

@ -76,22 +76,17 @@ class SideNav {
toggler.addEventListener('click', event => this.onMobileToggle(event)); toggler.addEventListener('click', event => this.onMobileToggle(event));
} }
if (window.innerWidth > 1200) { if (window.innerWidth >= 1200) {
if (this.state.get('pinned')) { if (this.state.get('pinned')) {
this.pin(); this.pin();
} } else {
if (!this.state.get('pinned')) {
this.unpin(); this.unpin();
} }
window.addEventListener('resize', () => this.onResize()); } else {
}
if (window.innerWidth < 1200) {
this.bodyRemove('hide'); this.bodyRemove('hide');
this.bodyAdd('hidden'); this.bodyAdd('hidden');
window.addEventListener('resize', () => this.onResize());
} }
window.addEventListener('resize', () => this.onResize());
this.base.addEventListener('mouseenter', () => this.onEnter()); this.base.addEventListener('mouseenter', () => this.onEnter());
this.base.addEventListener('mouseleave', () => this.onLeave()); this.base.addEventListener('mouseleave', () => this.onLeave());