Skip to content

fix(docs-site): configure base path for GitHub Pages#29

Merged
ryanbas21 merged 1 commit into
mainfrom
feat/docs-site
May 12, 2026
Merged

fix(docs-site): configure base path for GitHub Pages#29
ryanbas21 merged 1 commit into
mainfrom
feat/docs-site

Conversation

@ryanbas21
Copy link
Copy Markdown
Owner

Summary

  • Fix broken docs site deployment at ryanbas21.github.io/devtools/ — CSS, JS, and all navigation links were broken due to missing /devtools/ base path
  • Replace all hardcoded Attr.href strings with typed Route.link so links automatically respect the base path
  • Add CI post-build step to assemble correct deploy directory structure (work around elm-pages bug with script tag base path)

Test plan

  • Verify CI build passes
  • After deploy, check ryanbas21.github.io/devtools/ loads with correct styling
  • Verify navigation links (sidebar, header nav, package cards) all resolve correctly
  • Verify internal markdown links (e.g. Getting Started → Tree-Shaking guide) work
  • Test dark/light theme toggle still works
  • Test search functionality

🤖 Generated with Claude Code

The docs site was deploying with broken CSS, JS, and navigation because
all paths assumed root (/) but GitHub Pages serves at /devtools/.

- Add --base /devtools/ to elm-pages build for correct route generation
- Replace hardcoded Attr.href strings with Route.link typed routes
- Prefix markdown internal links and search URLs with Route.baseUrl
- Copy assets into devtools/ subdir and fix elm.js script path in CI
- Fix canonical URL from elm-pages.com to actual deployment URL
- Add style.css to public/ so Vite includes it in dist output

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ryanbas21 ryanbas21 merged commit 12f5b71 into main May 12, 2026
1 check passed
@ryanbas21 ryanbas21 deleted the feat/docs-site branch May 12, 2026 15:52
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.

1 participant