Skip to content

fix(desktop): show tooltips on all statusbar item variants - #49625

Closed
alelpoan wants to merge 2 commits into
NousResearch:mainfrom
alelpoan:fix/statusbar-tooltip
Closed

fix(desktop): show tooltips on all statusbar item variants#49625
alelpoan wants to merge 2 commits into
NousResearch:mainfrom
alelpoan:fix/statusbar-tooltip

Conversation

@alelpoan

Copy link
Copy Markdown
Contributor

What does this PR do?

Statusbar items (Terminal, YOLO, Gateway, Agents, Cron, version, timers) all had title strings defined in useStatusbarItems, but the title prop was never forwarded to the underlying HTML elements in StatusbarItemView. As a result, no tooltips appeared on hover making it hard to discover what each icon/button does without prior knowledge.
(e.g. the terminal toggle was undiscoverable without prior knowledge personally spent 20 minutes looking for it before finding it via the agent)

Related Issue

Fixes #

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 🔒 Security fix
  • 📝 Documentation update
  • ✅ Tests (adding or improving test coverage)
  • ♻️ Refactor (no behavior change)
  • 🎯 New skill (bundled or hub)

Changes Made

  • apps/desktop/src/app/shell/statusbar-controls.tsx: added
    title={item.title} to the <button> (action variant) and <div>
    (text variant) render branches in StatusbarItemView

How to Test

  1. Build and launch the desktop app
  2. Hover over any statusbar item (Terminal, YOLO, Agents, Cron, version,
    running timer, context usage, session timer)
  3. Verify a tooltip appears with a descriptive label
  4. Before this fix: no tooltips appeared on any statusbar item

Checklist

Code

  • I've read the Contributing Guide
  • My commit messages follow Conventional Commits
  • I searched for existing PRs to make sure this isn't a duplicate
  • My PR contains only changes related to this fix
  • I've run pytest tests/ -q and all tests pass
  • I've added tests for my changes
  • I've tested on my platform: Windows 10, Windows 11

Documentation & Housekeeping

  • I've updated relevant documentation (README, docs/, docstrings) — or N/A
  • I've updated cli-config.yaml.example if I added/changed config keys — or N/A
  • I've updated CONTRIBUTING.md or AGENTS.md if I changed architecture or workflows — or N/A
  • I've considered cross-platform impact (Windows, macOS) — tested on Windows 10 and Windows 11, macOS not verified
  • I've updated tool descriptions/schemas if I changed tool behavior — or N/A

Screenshots / Logs

Before
image

After
image
image

@alt-glitch alt-glitch added type/bug Something isn't working comp/tui Terminal UI (ui-tui/ + tui_gateway/) P3 Low — cosmetic, nice to have duplicate This issue or pull request already exists labels Jun 20, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #43077 — both forward title={item.title} to the statusbar render paths in StatusbarItemView (apps/desktop/src/app/shell/statusbar-controls.tsx) to surface native hover tooltips. #43077 is the earliest still-open PR with the identical mechanism (also Fixes #43071). Related: #41967 (Radix-tooltip approach for the same items).

@alelpoan alelpoan closed this Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tui Terminal UI (ui-tui/ + tui_gateway/) duplicate This issue or pull request already exists P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants