Conversation
✅ Deploy Preview for viteplus-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for viteplus-staging ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
![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: 10235b4ca5
ℹ️ 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".
| - - script | ||
| - id: home-theme-init | ||
| - | | ||
| document.documentElement.setAttribute('data-theme', 'light'); |
There was a problem hiding this comment.
The injected home-page script permanently sets document.documentElement[data-theme='light'], but this commit also removed the route-change cleanup that previously called root.removeAttribute('data-theme') when leaving layout: home. In SPA navigation (load / first, then open any docs page), the root stays forced to light, so dark-mode CSS like :root.dark:not([data-theme]) in docs/.vitepress/theme/styles.css no longer applies and non-home pages render with the wrong theme variables.
Useful? React with 👍 / 👎.
|
this is a fix in the oss header |
1 participant
Fixes:
Recording.2026-03-13.155344.2.mp4