chore(sidebar): use chevron for dropdown#8997
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview The icon uses new Reviewed by Cursor Bugbot for commit bfcfe67. Bugbot is set up for automated code reviews on this repo. Configure here. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8997 +/- ##
==========================================
+ Coverage 75.39% 75.41% +0.02%
==========================================
Files 98 98
Lines 8636 8636
Branches 318 318
==========================================
+ Hits 6511 6513 +2
+ Misses 2121 2119 -2
Partials 4 4 ☔ View full report in Codecov by Harness. |
There was a problem hiding this comment.
Pull request overview
Updates the SidebarGroup UI so expandable sidebar subgroups use a consistent chevron icon indicator (instead of the native disclosure marker), matching the visual affordance used by other dropdown-like components in ui-components.
Changes:
- Add a
ChevronDownIconto subgroup<summary>elements inSidebarGroup. - Style the chevron (size/color) and rotate it when the
<details>is open.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/ui-components/src/Containers/Sidebar/SidebarGroup/index.tsx | Renders a chevron icon alongside subgroup labels in the <summary>. |
| packages/ui-components/src/Containers/Sidebar/SidebarGroup/index.module.css | Adds chevron styling and rotates it in the [open] state. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Lighthouse Results
|
cc @ovflowd @bjohansebas based on https://openjs-foundation.slack.com/archives/CVAMEJ4UV/p1783185935444279