docs: Update URL structure #9237
Open
+1,068
−1,007
Conversation
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
|
Build successful! 🎉 |
reidbarber
reviewed
Nov 25, 2025
Member
reidbarber
left a comment
reidbarber
left a comment
There was a problem hiding this comment.
Looks like the index links are broken
|
Build successful! 🎉 |
# Conflicts: # packages/dev/s2-docs/pages/react-aria/GridList.mdx # packages/dev/s2-docs/pages/react-aria/Table.mdx # packages/dev/s2-docs/pages/react-aria/Tree.mdx # packages/dev/s2-docs/pages/s2/CardView.mdx # packages/dev/s2-docs/pages/s2/TableView.mdx # packages/dev/s2-docs/pages/s2/TreeView.mdx # packages/dev/s2-docs/src/Header.tsx # packages/dev/s2-docs/src/NavigationSuspense.tsx
|
Build successful! 🎉 |
devongovett
changed the title
|
Build successful! 🎉 |
devongovett
commented
Nov 26, 2025
| // eslint-disable-next-line | ||
| import {transformAsync} from '@parcel/rust/lib/index.js'; | ||
|
|
||
| export const getPages = cache(async () => { |
Member
Author
There was a problem hiding this comment.
This parses all MDX files so we have a complete list of pages, even when the Parcel build doesn't include all of them (building only S2 or RAC). Using an internal Parcel utility here for performance (the pages get parsed in parallel in Rust). Will have to keep this up to date with Parcel upgrades. Ideally there'd be a public utility for this.
reidbarber
previously approved these changes
Nov 26, 2025
# Conflicts: # packages/dev/s2-docs/pages/s2/index.mdx # packages/dev/s2-docs/src/Layout.tsx # packages/dev/s2-docs/src/MobileSearchMenu.tsx # packages/dev/s2-docs/src/SearchMenu.tsx
|
Build successful! 🎉 |
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.
Removes the
.htmlextension from all docs URLs. The CDN will automatically return either HTML or Markdown depending on theAcceptheader, and the URLs also look a bit cleaner this way.Updates the build to split the URL structure into separate domains for react-aria and react-spectrum. Locally, by default, there is still only a single server so
http://localhost:1234/s2andhttp://localhost:1234/react-aria(internationalized moved under react-aria). But in the build (or optionally locally), there are two separate Parcel builds for these, and they cross-link to each other. The list of pages has their URLs normalized so we always have absolute URLs with the correct domains attached according to the environment.