improvement(docs): align docs UI with the platform emcn design system#4962
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Tokens & global styling: Adds mirrored New chip UI: Introduces docs-local Layout polish: Footer prev/next loses vertical dividers in favor of one top rule and rounded hover rows; navbar/search drop bespoke dark search-trigger overrides removed in favor of tokenized search styling. Reviewed by Cursor Bugbot for commit a4db3f2. Configure here. |
Greptile SummaryThis PR aligns the docs app visual chrome with the platform's emcn design system by mirroring CSS tokens, introducing local
Confidence Score: 5/5Safe to merge — all changes are docs-app styling only, no functional logic or data paths are touched. The PR is a pure design-system alignment across the docs app: new CSS custom properties, two new UI components (Chip/ChipDropdown), and token-reference updates across existing components. No business logic, data fetching, auth, or shared library code is changed. The global.css token additions are deliberate and isolated to the docs app. Manual testing across both themes was confirmed by the author, and lint + type-check pass. No files require special attention. The new chip components are self-contained and well-typed, and the global.css token block is the only global-scope change. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["global.css\n:root / .dark token blocks\n--bg, --surface-*, --border-*, --text-*, --badge-*"] --> B["chip.tsx\nchipVariants (CVA)\nChip / ChipLink / ChipChevronDown"]
A --> C["chip-dropdown.tsx\nChipDropdown"]
A --> D["dropdown-menu.tsx\nCONTENT_BASE_CLASSES\nDropdownMenuItem"]
B --> E["navbar.tsx\nChipLink variant='brand'"]
B --> F["page-actions.tsx\nChip + leftIcon"]
C --> G["language-dropdown.tsx\nChipDropdown"]
D --> C
A --> H["page-type-badge.tsx\nbadge-* tokens"]
A --> I["sidebar-components.tsx\nsurface-* / text-* tokens"]
A --> J["page-footer.tsx\nborder / surface-3 tokens"]
A --> K["search-trigger.tsx\nsurface-5 / border-1 / font-season"]
A --> L["theme-toggle.tsx\nsurface-active / text-icon tokens"]
M["season.ts\nnext/font/local\n--font-season var"] --> N["layout.tsx\nhtml className += season.variable"]
N --> K
N --> B
Reviews (2): Last reviewed commit: "improvement(docs): build language dropdo..." | Re-trigger Greptile |
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit a4db3f2. Configure here.
Summary
Chip/ChipLink,ChipDropdown, emcn-styleDropdownMenu) and migrate the navbar Get started CTA, language dropdown, theme toggle, search trigger, copy-page button, and page navigation onto them--badge-*)--wp-*palette to the platform tokens so values can't driftType of Change
Testing
Tested manually — verified docs pages, API reference, and generated OpenAPI endpoint pages render in both themes;
bun run lintandtsc --noEmitpassChecklist