Docs clean up#704
Merged
Merged
Conversation
Server-render the GitHub stars badge (async component with Next revalidation) to avoid client flash and cache counts for 1 hour. Move the theme toggle out of the top SiteNav into the SiteFooter and add a footer Discord social link. Introduce an actions slot on SiteNav and pass the GitHubStarsBadge into multiple pages (home, blog, docs, brand, primitives, openclaw, telemetry). Simplify the /blog index to a compact title-and-author list with metadata and move the RSS link into the header. Add/update related CSS modules for blog, github badge, and footer. Update trajectories metadata with decision entries and lastUpdated timestamp.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
🟡 Deleted .blogBottom CSS class still referenced in blog post page, breaking layout
The .blogBottom CSS class was removed from web/components/blog/blog.module.css in this PR, but web/app/blog/[slug]/page.tsx:240 still references styles.blogBottom. With CSS Modules, this resolves to undefined, so the <div> wrapping the RSS icon on every blog post page loses its styling (centering via display: flex; justify-content: center, the margin-top: 3rem, padding-top: 1.5rem, and the top border separator). The blog index page correctly removed its .blogBottom usage and moved the RSS link to the header, but the blog post page was not updated to match.
(Refers to line 240)
Prompt for agents
The CSS class `.blogBottom` was deleted from `web/components/blog/blog.module.css` but is still used at `web/app/blog/[slug]/page.tsx:240`. The blog index page removed its blogBottom usage entirely when the RSS link was moved into the header. The blog post page should either: (1) remove the `.blogBottom` wrapper div entirely if the RSS icon at the bottom of blog posts is no longer desired, or (2) re-add the `.blogBottom` CSS class to blog.module.css if the RSS icon should remain styled at the bottom of blog posts. Looking at the blog index page change for reference, the intent seems to be to keep an RSS icon on individual post pages but the container class was accidentally deleted.
Was this helpful? React with 👍 or 👎 to provide feedback.
Contributor
|
Preview deployed!
This preview will be cleaned up when the PR is merged or closed. |
Delete web/public/docs static assets (favicon.png, logo/*) to prevent SST/OpenNext from treating the top-level public/docs directory as a static route prefix and causing preview environments to route /docs* to S3 instead of the Next app. Record this decision in .trajectories/active/traj_4r047wx9ml89.json (added decision entry) and update .trajectories/index.json lastUpdated timestamp.
# Conflicts: # .trajectories/completed/2026-04/traj_4r047wx9ml89.json # .trajectories/index.json
Add a web-only detection job to multiple GitHub workflows (node-compat, package-validation, rust-ci, test) and skip heavy jobs when the change set only touches web/ files. Move the docs navigation into the mobile hamburger: SiteNav now accepts mobileMenuContent and hideMobileDocsLink, DocsNav supports a mobileMenu variant with new styles, and the desktop docs sidebar is hidden on small screens. Add a /cloud footer link and update dark-mode footer styling (blue gradient and tinted divider). Introduce web/lib/content-paths.ts (resolveContentDir) and update web/lib/blog.ts, docs-markdown.ts, and docs.ts to resolve content at runtime; update next.config.mjs outputFileTracingIncludes to trace content/docs and content/blog to avoid build-machine absolute MDX paths. Add multiple .trajectories completed entries and update .trajectories/index.json. Remove the Waitlist resource from web/sst-env.d.ts.
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
1 participant
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.
Server-render the GitHub stars badge (async component with Next revalidation) to avoid client flash and cache counts for 1 hour. Move the theme toggle out of the top SiteNav into the SiteFooter and add a footer Discord social link. Introduce an actions slot on SiteNav and pass the GitHubStarsBadge into multiple pages (home, blog, docs, brand, primitives, openclaw, telemetry). Simplify the /blog index to a compact title-and-author list with metadata and move the RSS link into the header. Add/update related CSS modules for blog, github badge, and footer. Update trajectories metadata with decision entries and lastUpdated timestamp.