docs: align docs site with nexu design system#418
Conversation
- Remap VitePress theme tokens (brand colors, surfaces, text) from warm brown to nexu teal (#3DB9CE) - Replace default favicon logos with combined brand SVG (icon + text, 16px gap) for light/dark modes - Update font stack to Manrope/Inter (matching product UI) - Restyle .VPFeature cards to match nexu .card spec (border-radius 16px, subtle shadow, hover lift) - Add GitHubStars.vue: nav bar badge showing live star count via GitHub API with sessionStorage cache - Hide default VPSocialLinks in favor of the integrated badge Made-with: Cursor
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughUpdated the VitePress site theme: changed brand color and logos, added webfont preloads, disabled site title, introduced three navbar controls (LanguageSwitcher, ThemeToggle, GitHubStars), added components for language/theme/stars, and overhauled theme CSS (colors, typography, shadows, feature card and footer styling). Changes
Sequence DiagramsequenceDiagram
participant GitHubStars as GitHubStars<br/>Component
participant Session as SessionStorage
participant GitHubAPI as GitHub REST<br/>API
participant Nav as Navbar<br/>Badge
GitHubStars->>Session: check `nexu-gh-stars`
alt cache hit
Session-->>GitHubStars: return formatted count
else cache miss
GitHubStars->>GitHubAPI: fetch /repos/nexu-io/nexu
GitHubAPI-->>GitHubStars: return JSON with stargazers_count
GitHubStars->>GitHubStars: format count (e.g., 1.2k)
GitHubStars->>Session: store formatted count
end
GitHubStars->>Nav: render badge (icon + count)
Nav-->>GitHubStars: displayed
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying nexu-docs with
|
| Latest commit: |
b590dc0
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://393df0b7.nexu-docs.pages.dev |
| Branch Preview URL: | https://docs-design-system-alignment.nexu-docs.pages.dev |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
docs/.vitepress/theme/custom.css (1)
109-111: Avoid hiding the entire social-links container.This selector removes all social links, not just GitHub. Prefer targeting only the GitHub item (or removing GitHub from
themeConfig.socialLinksdirectly).Example CSS narrowing
-.VPSocialLinks { +.VPSocialLinks a[href*="github.com/nexu-io/nexu"] { display: none !important; }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/.vitepress/theme/custom.css` around lines 109 - 111, The CSS rule currently hides the entire .VPSocialLinks container which removes all social icons; instead narrow the change to only the GitHub item by either removing GitHub from themeConfig.socialLinks or replacing the blanket selector .VPSocialLinks with a selector that targets only the GitHub social item (e.g., the specific class/attribute used for the GitHub link within the social links list) so other social icons remain visible; update the CSS or themeConfig accordingly and keep .VPSocialLinks untouched.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/.vitepress/theme/custom.css`:
- Around line 61-71: The font-family declarations incorrectly quote generic
single-word family names; update both occurrences so Manrope and Inter are
unquoted (replace 'Manrope' and 'Inter' with Manrope and Inter) while leaving
multi-word families like 'PingFang SC' quoted; change the top declaration and
the block for .VPNavBarTitle .title, .VPHomeHero .name, .vp-doc h1, .vp-doc h2,
.vp-doc h3 to use unquoted Manrope and Inter for consistency with CSS
conventions.
In `@docs/.vitepress/theme/GitHubStars.vue`:
- Around line 35-40: The link with class "gh-badge" in GitHubStars.vue has no
accessible name when the stars prop/computed value is not yet loaded (visible
content is aria-hidden SVGs); add an explicit accessible name by binding an
aria-label (e.g. aria-label using the same expression as the :title: stars ?
`${stars} stars on GitHub` : 'GitHub') or render a visually-hidden text node
inside the anchor when stars is falsy so screen readers always have a readable
label for the link.
---
Nitpick comments:
In `@docs/.vitepress/theme/custom.css`:
- Around line 109-111: The CSS rule currently hides the entire .VPSocialLinks
container which removes all social icons; instead narrow the change to only the
GitHub item by either removing GitHub from themeConfig.socialLinks or replacing
the blanket selector .VPSocialLinks with a selector that targets only the GitHub
social item (e.g., the specific class/attribute used for the GitHub link within
the social links list) so other social icons remain visible; update the CSS or
themeConfig accordingly and keep .VPSocialLinks untouched.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 5e2a0cdd-8b15-418b-adbf-51f8817ba070
⛔ Files ignored due to path filters (2)
docs/public/favicon/nexu-logo-dark.svgis excluded by!**/*.svgdocs/public/favicon/nexu-logo-light.svgis excluded by!**/*.svg
📒 Files selected for processing (4)
docs/.vitepress/config.tsdocs/.vitepress/theme/GitHubStars.vuedocs/.vitepress/theme/custom.cssdocs/.vitepress/theme/index.ts
- Add custom LanguageSwitcher with fixed order and check-mark indicator - Replace appearance toggle slider with compact ThemeToggle icon button - Hide default VPNavBarTranslations and VPNavBarAppearance - Load JetBrains Mono for star count, reduce font size to 11px - Override VitePress shadow tokens (1–5) for lighter, more translucent feel - Soften VPFeature card shadows and hover effects - Restyle prev/next page links: smaller padding, arrow indicators, subdued text - Add aria-label to GitHubStars for accessibility - Unquote single-word font family names per CSS conventions Made-with: Cursor
Summary
#3DB9CE) design systemManrope / Intermatching the product UI.VPFeaturecards (border-radius 16px, subtle shadow, hover lift)GitHubStars.vue— live star count badge in nav bar via GitHub API withsessionStoragecacheVPSocialLinksicon in favor of the unified GitHub + star badgeChanged files
docs/.vitepress/config.ts— theme-color meta, logo paths, siteTitle: falsedocs/.vitepress/theme/custom.css— full CSS variable + component restylingdocs/.vitepress/theme/index.ts— Layout slot injection for GitHubStarsdocs/.vitepress/theme/GitHubStars.vue— new componentdocs/public/favicon/nexu-logo-light.svg— combined brand logo (dark text)docs/public/favicon/nexu-logo-dark.svg— combined brand logo (white text)Test plan
Made with Cursor
Summary by CodeRabbit
New Features
Style