Repository navigation
Move CSS and JS to src folders - #3277
Merged
Merged
Conversation
The CDN now serves fresh HTML, so the unbundled copies no longer need to
sit in public/ at their original /docs/css/ and /docs/js/ paths.
- public/docs/css/{main,vars}.css -> src/styles/
- public/docs/js/** -> src/scripts/
- Update the imports in HtmlHead, Search and the two Default layouts
- Drop the public/ globs from the prettier scripts (src/** covers them)
- Point the cspell ignore at src/scripts/**
Font Awesome CSS and webfonts stay in public/docs/css/fa so the webfonts
keep their stable, cacheable paths.
Done as pure git mv operations so other in-flight PRs on these files
rebase cleanly.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Pull request environment is available at https://stoctodocspr3277.z22.web.core.windows.net. You can view the ephemeral environment status in Octopus Deploy. This environment will be automatically deprovisioned when the pull request is closed, or after 7 days of inactivity. |
enf0rc3
marked this pull request as ready for review
August 4, 2026 03:50
borland
approved these changes
Aug 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes NES-257
What
The CDN now serves fresh HTML, so
main.css,vars.cssand the site JS can live insrc/and be bundled like everything else. Previously they sat inpublic/so clients holding HTML cached for up to 7 days could still resolve the original/docs/css/and/docs/js/paths.public/docs/css/{main,vars}.css→src/styles/public/docs/js/**→src/scripts/(joins the existingtheme-switcher.ts)HtmlHead.astro,Search.astro,src/layouts/Default.astroandsrc/themes/octopus/layouts/{Default,Search}.astropackage.jsonprettier scripts: dropped thepublic/docs/js/**andpublic/docs/css/**globs, sincesrc/**now covers these files. The only CSS left underpublic/docs/cssis the vendored Font Awesome bundle.cspell.jsonignore path moved frompublic/docs/js/**tosrc/scripts/**Font Awesome CSS and webfonts stay in
public/docs/css/fa— the@font-facerules invars.csspoint at them and the webfonts keep their stable, cacheable paths.public/css/report.cssalso stays put; the report pages link it by URL rather than importing it.Single commit, pure renames
Everything is one commit of
git mvoperations.git show --stat -Mshows all 27 moved files with 0 content changes, so other in-flight PRs touching these files rebase cleanly.Verification
pnpm buildsucceeds (2697 pages)/docs/_astro/components.BPDC2879.css,/docs/_astro/Default.astro_...DFIbMPWK.js, plus the dynamic-import chunks (code-blocks,detail-tabs,figures,headers,search-dialog,synonyms,youtube)/docs/css/fa/css/*.cssstill emitted and linked/docs/css/main.cssand/docs/js/no longer emitted, which is the intent of this changeNote:
prettier --checkreports pre-existing formatting drift acrosssrc/(including files untouched here), so I did not run--write— that would have buried the renames.🤖 Generated with Claude Code