Skip to content

Commit 89212d1

Browse files
committed
Fix toggling of sidenav dropdowns
1 parent 79fb475 commit 89212d1

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

src/_sass/_dash.scss

-6
Original file line numberDiff line numberDiff line change
@@ -389,12 +389,6 @@ $dash-max-width: 1330px;
389389
}
390390
}
391391

392-
.nav-link {
393-
&:focus {
394-
outline: $dash-highlight auto 2px;
395-
}
396-
}
397-
398392
a.frontanchor {
399393
padding-top: 50px;
400394
}

src/content/assets/js/main.js

+15-9
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,13 @@ function handleSearchShortcut(event) {
3232
}
3333
}
3434

35-
function scrollSidenavIntoView() {
35+
function setupSidenav() {
3636
const sidenav = document.getElementById('sidenav');
3737
if (!sidenav) {
3838
return;
3939
}
4040

41+
// Scroll active entry in to view.
4142
const activeEntries = sidenav.querySelectorAll('a.nav-link.active');
4243
if (activeEntries.length > 0) {
4344
const activeEntry = activeEntries[activeEntries.length - 1];
@@ -145,15 +146,10 @@ function setupCopyButtons() {
145146
}
146147

147148
$(function() {
149+
setupSidenav();
148150
setupOsTabs();
149151
initCookieNotice();
150152

151-
// Sidenav
152-
$('#sidenav i').on('click', function (e) {
153-
e.stopPropagation();
154-
$(this).parent('li').toggleClass('active');
155-
});
156-
157153
// open - close mobile navigation
158154
$('#menu-toggle').on('click', function (e) {
159155
e.stopPropagation();
@@ -170,8 +166,6 @@ $(function() {
170166
});
171167
}
172168

173-
scrollSidenavIntoView();
174-
175169
// Collapsible inline TOC expand/collapse
176170
$(".site-toc--inline__toggle").on('click', function () {
177171
var root = $("#site-toc--inline");
@@ -180,6 +174,18 @@ $(function() {
180174

181175
document.addEventListener('keydown', handleSearchShortcut);
182176

177+
const toggles = document.querySelectorAll('.nav-link.collapsible');
178+
toggles.forEach(function (toggle) {
179+
toggle.addEventListener('click', (e) => {
180+
if (toggle.classList.toggle('collapsed')) {
181+
toggle.ariaExpanded = 'false';
182+
} else {
183+
toggle.ariaExpanded = 'true';
184+
}
185+
e.preventDefault();
186+
});
187+
});
188+
183189
createGallery(
184190
'galleryOne',
185191
'galleryTwo',

0 commit comments

Comments
 (0)