Skip to content

fix(desktop): move new tab tooltip to top#775

Merged
Kitenite merged 1 commit intomainfrom
move-new-tab-tooltip
Jan 16, 2026
Merged

fix(desktop): move new tab tooltip to top#775
Kitenite merged 1 commit intomainfrom
move-new-tab-tooltip

Conversation

@Kitenite
Copy link
Copy Markdown
Collaborator

@Kitenite Kitenite commented Jan 16, 2026

Summary

  • Move the "New Tab" tooltip from bottom to top so it doesnt interfere with the dropdown

Test plan

  • Hover over the new tab button (+) in the tab strip
  • Verify the tooltip appears above the button instead of below

Note

Adjusts tooltip placement for the “New Tab” (+) button in GroupStrip.tsx.

  • Changes TooltipContent position from side="bottom" to side="top" for improved visibility in the tab strip

Written by Cursor Bugbot for commit 45dc7de. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

  • Bug Fixes
    • Adjusted tooltip positioning for the New Tab action to display above the button, improving visibility.

✏️ Tip: You can customize this high-level summary in your review settings.

Move the tooltip position from bottom to top for the new tab button
in the tab strip for better visibility.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 16, 2026

📝 Walkthrough

Walkthrough

A tooltip positioning adjustment in the GroupStrip component, changing the New Tab action hint display from below to above the trigger element for improved visual clarity.

Changes

Cohort / File(s) Summary
Tooltip Positioning
apps/desktop/src/renderer/screens/main/components/WorkspaceView/ContentView/TabsContent/GroupStrip/GroupStrip.tsx
Modified TooltipContent side prop from "bottom" to "top" for New Tab action hotkey hint display

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A tooltip takes flight, soaring high,
From bottom to top, reaching for the sky,
The hint for new tabs now greets from above,
A small tweak of polish, done with love! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The PR description covers the main objective but doesn't fully follow the provided template structure with all required sections. Expand the description to match the template: add a proper 'Description' section, specify 'Type of Change', complete the Testing section with detailed steps, and include 'Related Issues' and 'Additional Notes' sections as applicable.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: moving the new tab tooltip position from bottom to top.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@Kitenite Kitenite linked an issue Jan 16, 2026 that may be closed by this pull request
@Kitenite Kitenite merged commit 3b89b86 into main Jan 16, 2026
6 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

  • ⚠️ Neon database branch
  • ⚠️ Electric Fly.io app

Thank you for your contribution! 🎉

@Kitenite Kitenite deleted the move-new-tab-tooltip branch January 16, 2026 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] Tooltip for new tab blocking preset selection

1 participant