Skip to content

Commit 385352a

Browse files
committed
simply logic of activateCurrentPath function in nav script
1 parent cd6b78d commit 385352a

File tree

1 file changed

+7
-10
lines changed

1 file changed

+7
-10
lines changed

src/js/01-nav.js

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,8 @@
4545
})
4646

4747
nav.querySelector('[data-panel=explore] .context').addEventListener('click', function () {
48-
// NOTE logic assumes there are only two panels
4948
find(nav, '[data-panel]').forEach(function (panel) {
50-
panel.classList.toggle('is-active')
49+
panel.classList.toggle('is-active') // NOTE logic assumes there are only two panels
5150
})
5251
})
5352

@@ -97,14 +96,12 @@
9796
}
9897

9998
function activateCurrentPath (navItem, trace) {
100-
var ancestorClasses
101-
var ancestor = navItem.parentNode
102-
while (!(ancestorClasses = ancestor.classList).contains('nav-menu')) {
103-
if (ancestor.tagName === 'LI' && ancestorClasses.contains('nav-item')) {
104-
if (trace !== false) ancestorClasses.add('is-current-path')
105-
ancestorClasses.add('is-active')
106-
}
107-
ancestor = ancestor.parentNode
99+
var ancestorClassList
100+
var ancestor = navItem
101+
while ((ancestor = ancestor.parentNode) && ancestor !== menuPanel) {
102+
if (!(ancestorClassList = ancestor.classList).contains('nav-item')) continue
103+
if (trace !== false) ancestorClassList.add('is-current-path')
104+
ancestorClassList.add('is-active')
108105
}
109106
navItem.classList.add('is-active')
110107
}

0 commit comments

Comments
 (0)