Skip to content

fix(web): style sidebar action tooltips - #6371

Merged
maria-rcks merged 2 commits into
mainfrom
fix/sidebar-action-tooltips
Aug 14, 2026
Merged

fix(web): style sidebar action tooltips#6371
maria-rcks merged 2 commits into
mainfrom
fix/sidebar-action-tooltips

Conversation

@t3-code

@t3-code t3-code Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

What Changed

  • wrap the pinned, snooze, and settle sidebar quick actions in the app's styled Tooltip component
  • remove the pin button's native title tooltip
  • preserve each action's accessible name and existing behavior

Why

The three adjacent quick actions used inconsistent tooltip behavior: pin showed a browser-native tooltip, while snooze and settle showed none. They now share the same styled tooltip treatment.

UI Changes

before

before

after, light mode

unpin

light unpin

snooze

light snooze

settle

light settle

after, dark mode

unpin

dark unpin

snooze

dark snooze

settle

dark settle

Validation

  • vp run --filter @t3tools/web typecheck
  • vp lint apps/web/src/components/Sidebar.tsx --report-unused-disable-directives
  • vp fmt --check apps/web/src/components/Sidebar.tsx
  • verified in the running app in light and dark mode; all three styled popups render and all three trigger buttons have no native title

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for the UI changes
  • No video needed because this change does not alter motion

model: gpt-5.6-sol · harness: hermes agent

request provenance


Note

Low Risk
Presentation-only change in Sidebar quick-action buttons; no logic, API, or data handling changes.

Overview
Standardizes hover help on the sidebar’s snooze, unpin, and settle quick actions by wrapping each trigger in the shared Tooltip / TooltipPopup pattern already used elsewhere in Sidebar.tsx.

Snooze nests the tooltip around the popover trigger (clock icon) and adds a “Snooze thread” label. Unpin drops the native title attribute in favor of an “Unpin thread” styled tooltip while keeping aria-label. Settle gains a matching “Settle thread” tooltip on the card-row hover action. Click handlers, keyboard focus, and aria-label values are unchanged.

Reviewed by Cursor Bugbot for commit b67d8ad. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Add tooltips to snooze, unpin, and settle action buttons in the sidebar

Wraps the snooze, unpin, and settle buttons in Sidebar.tsx with Tooltip components, replacing the previous title attribute approach. Hovering or focusing each button now shows a labeled tooltip popup ('Snooze thread', 'Unpin thread', 'Settle thread').

Macroscope summarized b67d8ad.

Co-authored-by: Utkarsh Patil <73941998+UtkarshUsername@users.noreply.github.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Aug 12, 2026
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 11.3 KiB 11.3 KiB −5 B (−0.0%) 15.1 KiB
Codex Thread snapshot wire 5.5 KiB 5.5 KiB −1 B (−0.0%) 7.3 KiB
Codex Live turn WebSocket wire 5.9 KiB 5.9 KiB −4 B (−0.1%) 7.8 KiB
Codex Live turn WebSocket decoded 49.7 KiB 49.7 KiB 0 B (0.0%) 66.4 KiB
Codex Live turn messages 16 16 0 (0.0%) 21
Claude Total thread wire 11.3 KiB 11.3 KiB +19 B (+0.2%) 15.1 KiB
Claude Thread snapshot wire 5.5 KiB 5.5 KiB −1 B (−0.0%) 7.3 KiB
Claude Live turn WebSocket wire 5.8 KiB 5.9 KiB +20 B (+0.3%) 7.8 KiB
Claude Live turn WebSocket decoded 50.6 KiB 50.6 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 16 16 0 (0.0%) 21

Baseline: c9063f0 · PR result: b67d8ad · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 94.6 KiB
  • Claude decoded thread snapshot: 95.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

Co-authored-by: Utkarsh Patil <73941998+UtkarshUsername@users.noreply.github.com>
@maria-rcks maria-rcks closed this Aug 13, 2026
@maria-rcks maria-rcks reopened this Aug 14, 2026
@maria-rcks
maria-rcks enabled auto-merge (squash) August 14, 2026 19:39
@maria-rcks
maria-rcks merged commit 196c8ea into main Aug 14, 2026
26 checks passed
@maria-rcks
maria-rcks deleted the fix/sidebar-action-tooltips branch August 14, 2026 19:43
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 14, 2026
## What's Changed
* perf(desktop): speed up Windows update installation by @StiensWout in pingdotgg/t3code#6169
* fix(web): style sidebar action tooltips by @t3-code[bot] in pingdotgg/t3code#6371


**Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260814.1095...v0.0.34-nightly.20260814.1096

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260814.1096
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants