perf(homepage): split CSS by route group [Codex] - #1367
jderochervlk wants to merge 6 commits into
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
guide | 9604ead | Commit Preview URL Branch Preview URL |
Sep 20 2026, 02:45 PM |
jderochervlk
changed the title
Lighthouse baselineCompared with target branch
Download the full Lighthouse reports and baseline Commit |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
927ee00 to
3c5e854
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3c5e8546e6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
3c5e854 to
b209ae3
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b209ae3cb3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
b209ae3 to
3d20fe6
Compare
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
2c309ca to
cf2f948
Compare
cf2f948 to
7850e11
Compare
7850e11 to
bbd837e
Compare
Keep fonts, preflight and custom tokens shared while loading homepage, content and playground utilities through route links. Preserve responsive cascade ordering with shared shell sources, and prefetch documentation only on navigation intent. Reduce initial CSS from 71,764 to 45,176 raw bytes and from 13,099 to 10,366 gzip bytes. Record the deliberate additional stylesheet and 252-byte gzip route-module overhead. Cover cold routes, navigation, fonts, prefetch and playground themes without changing visual baselines.
Let head stylesheet links handle render blocking without hiding the entire document. Keep fallback content readable when fonts or shared CSS fail, and assert real applied styles in navigation coverage.
Preserve route stylesheet isolation, typography, navigation, foundation deduplication, and readable prerendered content without app hydration or available fonts/styles. Addresses PR #1355 comment 4054821928.
Include src/components/LandingPage modules in homepage utilities and exclude them from content utilities. Add emitted-CSS assertions covering the misplaced layout classes without changing component locations.
Point contributors to the route-shared Tailwind token and utility file introduced by the CSS boundary split.
Move the no-hydration and route stylesheet coverage into typed ReScript specs. Restrict the homepage Cypress config to generated .cy.jsx specs now that the JavaScript tests are gone.
bbd837e to
9604ead
Compare
1 participant
Continue #1354 on top of #1366 by keeping documentation and playground styles out of the initial homepage stylesheet. Tailwind source detection is explicit for each route group; a wrapper-free homepage layout and the existing content/playground routes own their stylesheet links.
Fonts, preflight and custom theme tokens remain shared. Default Tailwind values are inlined so route-only utilities do not depend on unused root tokens. Each route includes the shared navbar utility sources to preserve responsive cascade ordering. Documentation prefetch now starts on hover or focus intent instead of immediately entering the viewport.
Remove the legacy document-wide opacity gate and rely on normal head stylesheet blocking. Server-rendered content stays readable if the shared stylesheet fails, and homepage paragraphs render with fallback fonts without JavaScript.
Initial CSS falls from 71,764 to 45,161 raw bytes and from 13,099 to 10,362 gzip bytes. The route boundary adds one JavaScript request, one stylesheet request and 241 gzip bytes of JavaScript; the 415-element homepage DOM is unchanged. Lighthouse remains informational, with no score threshold.