Skip to content
This repository was archived by the owner on Aug 3, 2024. It is now read-only.

Commit ad31d96

Browse files
mpickeringbgamari
authored andcommitted
Revert "Fix #783 Don't show button if --quickjump not present"
This reverts commit 3ab18cf. This commit breaks the quickjump functionality on `hackage.haskell.org`. See https://gitlab.haskell.org/ghc/ghc/-/issues/21984
1 parent 86d5fce commit ad31d96

File tree

7 files changed

+3276
-2284
lines changed

7 files changed

+3276
-2284
lines changed

haddock-api/resources/html/haddock-bundle.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

haddock-api/resources/html/js-src/quick-jump.tsx

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -45,12 +45,11 @@ class PageMenuButton extends Component<any, any> {
4545

4646
}
4747

48-
function addSearchPageMenuButton(quickJumpButton: HTMLLIElement) {
49-
const showHide = function(action: () => void) {
50-
const pageMenu = document.querySelector('#page-menu') as HTMLUListElement;
51-
preact.render(<PageMenuButton onClick={action} title="Quick Jump" />, pageMenu, quickJumpButton);
52-
};
53-
return showHide;
48+
function addSearchPageMenuButton(action: () => void) {
49+
const pageMenu = document.querySelector('#page-menu') as HTMLUListElement;
50+
const dummy = document.createElement('li');
51+
pageMenu.insertBefore(dummy, pageMenu.firstChild);
52+
preact.render(<PageMenuButton onClick={action} title="Quick Jump" />, pageMenu, dummy);
5453
}
5554

5655
// -------------------------------------------------------------------------- //
@@ -411,14 +410,10 @@ function NoResultsMsg(props: { searchString: string }) {
411410
}
412411

413412
export function init(docBaseUrl?: string, showHide?: (action: () => void) => void) {
414-
// A <li> element with this id should exist if --quickjump was set
415-
const quickJumpButton = document.getElementById('quick-jump-button');
416-
if (quickJumpButton)
417-
preact.render(
418-
<QuickJump baseUrl={docBaseUrl || "."} showHideTrigger={showHide ||
419-
addSearchPageMenuButton(quickJumpButton as HTMLLIElement)} />,
420-
document.body
421-
);
413+
preact.render(
414+
<QuickJump baseUrl={docBaseUrl || "."} showHideTrigger={showHide || addSearchPageMenuButton} />,
415+
document.body
416+
);
422417
}
423418

424419
// export to global object

0 commit comments

Comments
 (0)