archive-gh-me/docsify
1
0
Fork 0
mirror of https://github.com/docsifyjs/docsify.git synced 2025-12-08 19:55:52 +00:00
Code Issues Projects Releases Wiki Activity

refactor: optimize toggleSidebar trigger conditions (#2557)

Browse Source
This commit is contained in:
Luffy 2025-06-14 20:56:57 +08:00 committed by GitHub
parent 3d53203f58
commit 71ac41e68e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/core/event/index.js
View File

@ -343,8 +343,7 @@ export function Events(Base) {
onNavigate(source) {
const { auto2top, topMargin } = this.config;
const { path, query } = this.route;
this.#markSidebarActiveElm();
const activeSidebarElm = this.#markSidebarActiveElm();
// Note: Scroll position set by browser on forward/back (i.e. "history")
if (source !== 'history') {
@ -371,13 +370,20 @@ export function Events(Base) {
}
}
const isNavigate = source === 'navigate';
const hasId = 'id' in query;
const noSubSidebar = !activeSidebarElm?.querySelector('.app-sub-sidebar');
// Clicked anchor link
if (path === '/' || (query.id && source === 'navigate')) {
isMobile() && this.#toggleSidebar(false);
const shouldCloseSidebar =
path === '/' || (isNavigate && (hasId || noSubSidebar));
if (shouldCloseSidebar && isMobile()) {
this.#toggleSidebar(false);
}
// Clicked anchor link or page load with anchor ID
if (query.id || source === 'navigate') {
if (hasId || isNavigate) {
this.#focusContent();
}
}