fix(web): replace the update tooltip with a changelog popover

[RakshithBhat03]

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 --check passed; local browser pass covered both update states, changelog scrolling, and direct sidebar download/restart-install actions. Web typecheck still reports two pre-existing conditionalUI errors in src/components/clerk/electronPasskeys.test.ts.

Screenshots

Update ready to download

Before	After

Update downloaded

Before	After

---

Built by GPT-5.6 Sol in T3 Code through the Codex harness.

Summary by CodeRabbit

• New Features
  • Nightly updates with available release notes now open an interactive popover for easier reading.
  • Release notes are displayed in a scrollable panel without leaving the current view.
  • Updates without release notes continue to provide a helpful fallback tooltip.
  • Update controls now support consistent hover, focus, click, and disabled states for improved accessibility and interaction.

---

Note

Low Risk
Sidebar Electron update UI and accessibility behavior only; no auth, API, or data-path changes.

Overview
Nightly desktop updates with release notes no longer cram the changelog into a tooltip. Eligible states now use a hover-activated popover with a scrollable changelog; stable channel or empty notes still get the original tooltip only.

The sidebar pill click behavior is unchanged in intent: click still downloads or installs immediately—preventBaseUIHandler stops the popover from toggling, and the popover shows status plus notes only (no duplicate action buttons). A new SidebarUpdateReleaseNotesPopover owns open state, hover delays, keyboard :focus-visible handling, and cancellation of premature hover-close while the trigger stays focused or hovered; when notes are ineligible it disables the popover and strips popover ARIA from the trigger.

SidebarUpdateReleaseNotes (renamed from the tooltip variant) is changelog-only—eligibility is gated upstream. Tests add a large hook-harness suite covering download-on-click, popover vs tooltip fallbacks, downloading with aria-disabled, and focus/hover edge cases.

^{Reviewed by Cursor Bugbot for commit 1a0c3c0. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Replace sidebar update tooltip with a changelog popover

• Introduces SidebarUpdateReleaseNotesPopover in SidebarUpdatePill.tsx that coordinates a Tooltip fallback with a hover/focus-driven Popover for release notes, using delayed hover open/close and keyboard focus latching
• SidebarUpdateControl now renders the popover only when eligible (showUpdateDetails, nightly channel, and non-empty release notes); clicking the button invokes the update action without toggling the popover
• Refactors SidebarUpdateReleaseNotes to always render the release notes markup and defer eligibility to the caller, moving width/scroll constraints into the popover
• Adds a Vitest suite in SidebarUpdatePill.test.tsx covering hover/focus behavior, state transitions, and popover eligibility
• Behavioral Change: popover ARIA semantics are stripped from the trigger when only the tooltip is eligible; tooltip is disabled when the popover is enabled

^{Macroscope summarized 1a0c3c0.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ae188458-6a75-4061-87fd-4eb5aae4e3ea

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7801935e-b64f-4eb8-9188-4bcf28733aec

📥 Commits

Reviewing files that changed from the base of the PR and between 11f0513 and 301fb4185ec63f53ae36e542010403c65bed50d3.

📒 Files selected for processing (2)

• apps/web/src/components/sidebar/SidebarUpdatePill.test.tsx
• apps/web/src/components/sidebar/SidebarUpdatePill.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

SidebarUpdatePill now displays nightly release notes in a hoverable popover. Other update states retain tooltip behavior. Tests cover rendering, focus, blur, clicks, downloads, and fallback content.

Changes

Sidebar update release notes

Layer / File(s)	Summary
Release notes presentation apps/web/src/components/sidebar/SidebarUpdatePill.tsx	The component adds a standalone release-notes panel, popover state, popover imports, and scroll-limited viewport styling.
Conditional trigger flow apps/web/src/components/sidebar/SidebarUpdatePill.tsx, apps/web/src/components/sidebar/SidebarUpdatePill.test.tsx	Nightly updates with release notes use the popover. Other states use the tooltip. Tests cover trigger actions, focus, blur, popover rendering, and fallback behavior.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to 301fb

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: stienswout, t3dotgg

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 2 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly summarizes the main change: replacing the update tooltip with a changelog popover.
Description check	✅ Passed	The description explains the change and rationale, documents UI changes with screenshots, and includes testing details and known pre-existing errors.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

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}

[RakshithBhat03]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[macroscopeapp]

Approvability

Verdict: Approved at 1a0c3c0

Macroscope's review found this PR approvable — This is a self-contained Electron sidebar UI replacement that moves existing nightly release notes from a tooltip into a bounded hover/focus popover while preserving update actions and fallback tooltips. The interaction behavior is covered by focused tests, with no backend, schema, security, or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

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}

[macroscopeapp]

One finding on the new focus-latch behavior in SidebarUpdatePill.tsx. Everything else (primitive usage, popover surface/geometry, viewport scroll ownership, eventDetails.cancel() pattern) matches the existing Popover conventions in this app.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Reviewed the release-notes popover migration for shared-primitive, theming, and interaction consistency. One new finding on the conditional trigger host; the previously raised popup theming difference between the popover and tooltip branches still appears unaddressed at this head.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

One finding on the unified trigger host: the always-mounted PopoverTrigger now advertises a dialog popup on the pill in states where the popover can never open. Details inline.

^{Posted via Macroscope — UI Consistency}

[cursor]

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 cddb765. Configure here.}

[t3dotgg]

Note

🤖 GPT-5.6 Sol responding on behalf of Theo

We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together.

This replaces a nightly-update tooltip with a 702-line hover popover for long changelogs. Download and restart remain direct actions, so the extra popover state, accessibility behavior, and tests are too much for optional release-note reading.

If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed.