fix(web): replace the update tooltip with a changelog popover - #7880
fix(web): replace the update tooltip with a changelog popover#7880RakshithBhat03 wants to merge 5 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthrough
ChangesSidebar update release notes
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This PR makes nightly release notes easier to read and scroll while preserving existing download and restart/install actions. No actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
One finding: the new hover-only release-notes popover removes keyboard access to the update details that the tooltip previously provided on focus. Details inline.
Posted via Macroscope — UI Consistency
|
@coderabbitai review |
✅ Action performedReview finished.
|
ApprovabilityVerdict: Skipped Macroscope did not run approvability analysis for this PR. Macroscope could not determine whether this PR modifies its approvability configuration, so the PR was not approved automatically. A PR that may change the rules that govern approval is never approved automatically. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1521d6d. Configure here.
There was a problem hiding this comment.
One finding: the release-notes popover branch drops the update-state surface treatment that the tooltip branch still applies for the same update states.
Posted via Macroscope — UI Consistency

Nightly update notes were rendered in a tooltip, so the panel disappeared when the pointer moved into it and long changelogs were difficult to read or scroll.
Nightly updates with release notes now use a hover popover that stays open while reading and scrolling. Compared with the superseded #7876 draft, the sidebar pill keeps its original direct action: clicking an available update immediately starts the download, while clicking a downloaded update invokes the existing restart/install flow. The popover contains only the update status and changelog, with no duplicate Download or Restart and install buttons.
Stable updates and states without release notes continue using the existing tooltip. The trigger remains a native button with its accessible label and visible focus treatment. The popover has a screen-reader title and does not move focus away from the trigger when it opens.
Tests: 42 focused tests, lint, format check, and
git diff --checkpassed; local browser pass covered both update states, changelog scrolling, and direct sidebar download/restart-install actions. Web typecheck still reports two pre-existingconditionalUIerrors insrc/components/clerk/electronPasskeys.test.ts.Screenshots
Update ready to download
Update downloaded
Built by GPT-5.6 Sol in T3 Code through the Codex harness.
Summary by CodeRabbit
Note
Low Risk
UI-only sidebar interaction change with no auth, data, or update-backend logic. Focus/hover close handling is the main behavioral risk.
Overview
Nightly updates with release notes no longer live in a tooltip that closes when the pointer moves into it. They now open as a hover popover so the changelog can be read and scrolled.
The sidebar pill still runs download or restart/install on click. The popover is changelog-only (no duplicate action buttons), stays open while hovered or focused, and does not steal focus. Stable updates and empty notes keep the existing tooltip.
Reviewed by Cursor Bugbot for commit da6109b. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Replace sidebar update tooltip with a hover-driven changelog popover
SidebarUpdateReleaseNotesPopover) instead of embedding rich content inside the tooltip.SidebarUpdateReleaseNotesis renamed fromSidebarUpdateReleaseNotesTooltipand no longer applies its own max-width or scrolling — the popover handles sizing and viewport constraints.SidebarUpdateControlgates the popover onshowReleaseNotesPopover(nightly + non-empty release notes); all other builds fall back to a plain-textTooltip.onBluronly closes when neither hover nor focus remains inside the popover.Tooltipno longer renders release-notes markup; any code expecting rich content inside the tooltip will see plain text only. Reviewers should checkSidebarUpdateControltrigger handlers andSidebarUpdateReleaseNotescallers.Macroscope summarized da6109b.