Skip to content

Commit 66cfebc

Browse files
committedFeb 23, 2024
fix mobile nav bar not working on introduction and libraries pages
1 parent 60ac8b5 commit 66cfebc

File tree

6 files changed

+19
-16
lines changed

6 files changed

+19
-16
lines changed
 

Diff for: ‎src/website/dom-elements-common.js

+9-2
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,9 @@
1-
export const codeElements = document.querySelectorAll('.plain-text pre code');
2-
export const counterElement = document.querySelector('.counter');
1+
export const codeElements = document.querySelectorAll(".plain-text pre code");
2+
export const counterElement = document.querySelector(".counter");
3+
4+
export const navbarElement = document.querySelector("nav.navbar");
5+
export const menuTriggerElement = document.querySelector(".menu-trigger");
6+
export const menuLinks = document.querySelectorAll(".navbar .menu a");
7+
export const menuScrollableLinks = navbarElement.querySelectorAll("a.scrollto");
8+
9+
export const sectionElements = document.getElementsByTagName("section");

Diff for: ‎src/website/dom-elements.js

+1-9
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,6 @@
11
export * from '../dom-elements.js';
22
export * from './dom-elements-common.js';
33

4-
export const navbarElement = document.querySelector('nav.navbar');
5-
export const menuTriggerElement = document.querySelector('.menu-trigger');
6-
export const menuLinks = document.querySelectorAll('.navbar .menu a');
7-
export const menuScrollableLinks =
8-
navbarElement.querySelectorAll('a.scrollto');
9-
10-
export const sectionElements = document.getElementsByTagName('section');
11-
124
export const extensionSection = document.querySelector('.update-site');
135
export const ebookSection = document.querySelector('.jtw-ebook-banner');
146

@@ -22,4 +14,4 @@ export const debuggerSection = document.getElementById('debugger-io');
2214
export const shareJwtButton = document.querySelector('.website-share button');
2315
export const shareJwtTextElement = shareJwtButton.querySelector('span');
2416

25-
export const starsElements = document.getElementsByClassName('stars');
17+
export const starsElements = document.getElementsByClassName('stars');

Diff for: ‎src/website/introduction/index.js

+2
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
import { CCPAModal } from "../ccpa-modal.js";
22
import { setupJwtCounter } from "../counter.js";
33
import { setupHighlighting } from "../highlighting.js";
4+
import { setupNavbar } from "../navbar.js";
45
import { TopBanner } from "../top-banner.js";
56

67
// Initialization
8+
setupNavbar();
79
setupHighlighting();
810
setupJwtCounter();
911
CCPAModal();

Diff for: ‎src/website/libraries/index.js

+2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
import { CCPAModal } from "../ccpa-modal.js";
22
import { setupJwtCounter } from "../counter.js";
33
import { setupHighlighting } from "../highlighting.js";
4+
import { setupNavbar } from "../navbar.js";
45
import { TopBanner } from "../top-banner.js";
56
import { setupLibraries } from "./libraries.js";
67

78
// Initialization
9+
setupNavbar();
810
setupLibraries();
911
setupHighlighting();
1012
setupJwtCounter();

Diff for: ‎src/website/navbar.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
import { getOffsetBoundingClientRect } from './utils.js';
22

3-
import {
3+
import {
44
navbarElement,
55
menuTriggerElement,
66
menuLinks,
77
sectionElements,
88
menuScrollableLinks
9-
} from './dom-elements.js';
9+
} from './dom-elements-common.js';
1010

1111
export function setupNavbar() {
1212
window.addEventListener('scroll', () => {
@@ -50,4 +50,4 @@ export function setupNavbar() {
5050
document.body.classList.remove('menu-mobile');
5151
});
5252
});
53-
}
53+
}

Diff for: ‎src/website/smooth-scrolling.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { menuScrollableLinks, navbarElement } from './dom-elements.js';
1+
import { menuScrollableLinks, navbarElement } from './dom-elements-common.js';
22
import { isWideScreen } from '../utils.js';
33

44
import $ from 'jquery';
@@ -9,7 +9,7 @@ import log from 'loglevel';
99
// jQuery somewhere else.
1010
export function smoothScrollTo(element) {
1111
// TODO: don't use jQuery
12-
12+
1313
const navHeight = $(navbarElement).height();
1414
const targetElement = $(element);
1515

0 commit comments

Comments
 (0)
Please sign in to comment.