Skip to content

Commit 60a474c

Browse files
committed
harden code around menu dispaly or not
1 parent f1b9c4c commit 60a474c

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

src/mainPage/menu.ts

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,6 @@ const ensureMenuSkeleton = () => {
7272
navMenu.className = 'app-nav'
7373
navMenu.setAttribute('aria-label', 'App navigation')
7474
navMenu.hidden = true
75-
navMenu.style.display = 'none'
7675

7776
const headerEl = document.createElement('div')
7877
headerEl.className = 'menu-header'
@@ -293,17 +292,25 @@ export const refreshMenu = (layout: 'mobile' | 'desktop') => {
293292
navMenu.classList.add('mobile-hidden')
294293
navMenu.classList.remove('mobile-visible')
295294
toggle.hidden = false
295+
toggle.style.display = ''
296296
collapseBtn.hidden = true
297+
collapseBtn.style.display = 'none'
297298
overlay.hidden = true
299+
overlay.style.display = 'none'
298300
navMenu.hidden = false
301+
navMenu.style.display = ''
299302
navMenu.classList.remove('collapsed')
300303
toggle.setAttribute('aria-expanded', 'false')
301304
} else {
302305
navMenu.classList.remove('mobile-hidden', 'mobile-visible')
303306
toggle.hidden = true
307+
toggle.style.display = 'none'
304308
collapseBtn.hidden = false
309+
collapseBtn.style.display = ''
305310
overlay.hidden = true
311+
overlay.style.display = 'none'
306312
navMenu.hidden = false
313+
navMenu.style.display = ''
307314
applyMenuCollapsedState(navMenu)
308315
updateCollapseButtonPosition(navMenu, collapseBtn)
309316
toggle.setAttribute('aria-expanded', 'false')
@@ -321,16 +328,20 @@ export const createLeftSideMenu = async (subject: NamedNode, outliner: OutlineMa
321328
if (!navMenu || !menuToggle || !menuOverlay) return
322329
navMenu.classList.remove('mobile-visible')
323330
navMenu.classList.add('mobile-hidden')
331+
navMenu.style.display = ''
324332
menuToggle.setAttribute('aria-expanded', 'false')
325333
menuOverlay.hidden = true
334+
menuOverlay.style.display = 'none'
326335
}
327336

328337
const openMobileMenu = () => {
329338
if (!navMenu || !menuToggle || !menuOverlay) return
330339
navMenu.classList.remove('mobile-hidden')
331340
navMenu.classList.add('mobile-visible')
341+
navMenu.style.display = ''
332342
menuToggle.setAttribute('aria-expanded', 'true')
333343
menuOverlay.hidden = false
344+
menuOverlay.style.display = ''
334345
}
335346

336347
const collapseBtn = document.getElementById('MenuCollapseBtn') as HTMLButtonElement | null

0 commit comments

Comments
 (0)