Fix nav scripts: remove dead interceptors, debounce observer, scope n…#261
Fix nav scripts: remove dead interceptors, debounce observer, scope n…#261Blargian wants to merge 1 commit into
Conversation
…avbar hide
- ask-ai-button.js: remove capture-phase click/mousedown/keydown
interceptors for Mintlify native search ('Can you tell me about')
since we replaced native search with Inkeep
- tab-nav.js: debounce MutationObserver via RAF so rapid React
re-renders batch into one flush instead of thrashing main thread
with repeated querySelectorAll calls
- styles.css: scope navbar opacity-hide to .ch-homepage only so
SPA page navigation doesn't trigger the fade-in on every page
Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 208f6fb. Configure here.
| patchTabButtons(); | ||
| styleDropdownHeaders(); | ||
| markNavbarReady(); | ||
| }); |
There was a problem hiding this comment.
Delayed homepage class causes flicker
Medium Severity
The mutation observer now runs applyHomepageClass inside a debounced requestAnimationFrame, but navbar opacity hiding is scoped to .ch-homepage. Client-side route changes can paint at least one frame where the URL and ch-homepage disagree, so the homepage navbar may flash visible or stay hidden on the wrong page.
Reviewed by Cursor Bugbot for commit 208f6fb. Configure here.
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
1 participant
…avbar hide
ask-ai-button.js: remove capture-phase click/mousedown/keydown interceptors for Mintlify native search ('Can you tell me about') since we replaced native search with Inkeep
tab-nav.js: debounce MutationObserver via RAF so rapid React re-renders batch into one flush instead of thrashing main thread with repeated querySelectorAll calls
styles.css: scope navbar opacity-hide to .ch-homepage only so SPA page navigation doesn't trigger the fade-in on every page