-
Notifications
You must be signed in to change notification settings - Fork 538
[TOOL-2816] Nebula - update sidebar + other minor UI adjustments #5823
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
c823375
to
30a580f
Compare
30a580f
to
d0a2a6c
Compare
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5823 +/- ##
=======================================
Coverage 54.11% 54.11%
=======================================
Files 1102 1102
Lines 59202 59202
Branches 4885 4885
=======================================
Hits 32037 32037
Misses 26441 26441
Partials 724 724
*This pull request uses carry forward flags. Click here to find out more. |
Merge activity
|
## Problem solved Short description of the bug fixed or feature added <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on improving the UI components in the `ChatBar`, `ChatSidebar`, and `ContextFilters` by updating styles, modifying button variants, and enhancing layout structures. It also introduces a new `SidebarIconLink` component for better organization. ### Detailed summary - Removed `variant="primary"` from `Button` in `ChatBar`. - Simplified `Button` in `NebulaMobileNav`. - Adjusted class names in `ChatHistoryPage` for responsiveness. - Updated `Badge` components in `ContextFilters` to use `variant="secondary"`. - Replaced multiple icons in `ChatSidebar` with new ones. - Changed `Button` variant to `outline` in `ChatSidebar`. - Added new `SidebarIconLink` component for navigation links. - Adjusted layout spacing in `ChatSidebar`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
36ae01a
to
1e27491
Compare
d0a2a6c
to
9bd9923
Compare
Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR focuses on improving the UI components in the
ChatBar
,ChatSidebar
, andContextFilters
by refining button styles, adjusting layout classes, and enhancing accessibility features. It also introduces new components and modifies existing ones for better consistency.Detailed summary
variant="primary"
fromButton
inChatBar.tsx
.Button
usage inNebulaMobileNav.tsx
.ChatHistoryPage.tsx
for better layout.variant="secondary"
toBadge
components inContextFilters.tsx
.ChatSidebar.tsx
and changed button styles.SidebarIconLink
component for cleaner link rendering.