-
Notifications
You must be signed in to change notification settings - Fork 352
docs: fix homepage title duplication and disable glow animation on mobile for all themes #22675
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -1604,7 +1604,7 @@ main, | |||||
| } | ||||||
|
|
||||||
| /* Disable heavy background animation on mobile to reduce CPU/battery usage */ | ||||||
| :root[data-theme='light'] body::before { | ||||||
| body::before { | ||||||
|
||||||
| body::before { | |
| :root[data-theme] body::before { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing the homepage
titletoHomefixes the duplicated document title, but it also changes howdocs/src/components/CustomHead.astrodetects the homepage (isHomePage = rawPageTitle === siteTitle). Withtitle: Home, the homepage will now be treated as a non-home page and OG/twitter titles will becomeHome | GitHub Agentic Workflowsinstead of just the site title. Consider updating the home-page detection inCustomHead.astro(e.g., use the pathname) or adding an explicit frontmatter flag/value for the homepage OG title.