refactor(local): move the update prompt from a banner to a title bar button - #1778
Conversation
|
@AnishSarkar22 is attempting to deploy a commit to the Rohan Verma's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Description
Motivation and Context
FIX #
Screenshots
API Changes
Change Type
Testing Performed
Checklist
High-level PR Summary
This PR refactors the update notification UI from a full-width banner into a compact icon button in the title bar. The changes introduce a new
--noticecolor variable to the design system for non-destructive attention-grabbing elements, replace theUpdateBannercomponent with anUpdateButtonthat appears in the dashboard's title bar controls, and update corresponding tests to reflect the new behavior. The update notification is now displayed as a ghost button with a download icon that only appears when an update is ready to install.⏱️ Estimated Review Time: 15-30 minutes
💡 Review Order Suggestion
.cursor/skills/color-system/PALETTE.csssurfsense_local/frontend/src/index.csssurfsense_local/frontend/src/components/ui/icons.tsxsurfsense_local/frontend/src/features/updates/update-settings.tsxsurfsense_local/frontend/src/features/dashboard/dashboard-page.tsxsurfsense_local/frontend/src/app/app-bootstrap.tsxsurfsense_local/frontend/src/features/updates/update-settings.test.tsxsurfsense_local/frontend/src/test-utils.tsx