Skip to content

Feat/pagefind bis#70

Merged
rubensworks merged 3 commits intocomunica:masterfrom
jitsedesmet:feat/pagefind-bis
Apr 15, 2026
Merged

Feat/pagefind bis#70
rubensworks merged 3 commits intocomunica:masterfrom
jitsedesmet:feat/pagefind-bis

Conversation

@jitsedesmet
Copy link
Copy Markdown
Member

Fresh take on #68

jitsedesmet and others added 2 commits April 14, 2026 13:34
- Install pagefind@1.5.2 as a dev dependency
- Update build script to run pagefind after next export (outputs to out/pagefind)
- Add pagefind-component-ui CSS/JS to _document.js; place <pagefind-modal>
  in the page body so it is available site-wide
- Desktop: search trigger button (with 'Search…' text and ⌘K/Ctrl+K badge)
  appears at the right end of the nav bar via .nav-search-desktop
- Mobile (≤1020px): compact search icon shown next to the hamburger toggle;
  desktop trigger is hidden
- Ctrl+K / Cmd+K keyboard shortcut opens the modal from anywhere on the page
- Keyboard shortcut badges styled as white pills with brand-red text so they
  are readable on the red nav background
- Docs pages get data-pagefind-weight="2" on their content wrapper so /docs/
  results rank above equally-relevant non-docs pages in search
- Override --pf-outline-focus and --pagefind-ui-primary to Comunica red
- Add z-index: 100 to the fixed nav (modal sits at 9999)
- Gitignore public/pagefind/ (generated at build time, not to be committed)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jitsedesmet
Copy link
Copy Markdown
Member Author

image image

@jitsedesmet
Copy link
Copy Markdown
Member Author

image image

for mobile

Comment thread package.json Outdated
@jitsedesmet jitsedesmet requested a review from rubensworks April 15, 2026 10:46
@jitsedesmet
Copy link
Copy Markdown
Member Author

I also removed the package annotation since it said 'yarn' even though we have a package-lock.json (npm)

@rubensworks rubensworks merged commit b3dfbd7 into comunica:master Apr 15, 2026
1 check passed
@rubensworks
Copy link
Copy Markdown
Member

Very nice, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants