Skip to content

simplify the tooltip#566

Merged
Kitenite merged 2 commits intomainfrom
simplify-the-tooltip
Jan 1, 2026
Merged

simplify the tooltip#566
Kitenite merged 2 commits intomainfrom
simplify-the-tooltip

Conversation

@Kitenite
Copy link
Copy Markdown
Collaborator

@Kitenite Kitenite commented Jan 1, 2026

Summary by CodeRabbit

  • New Features

    • Introduced a two-part create control: a primary "Create" action button for immediate workspace creation plus a separate "More create options" dropdown with alternative creation choices.
  • Style

    • Simplified tooltip to show plain "New workspace" text.
    • Updated layout and icon sizing for the combined control to improve spacing and visual clarity.

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 1, 2026

📝 Walkthrough

Walkthrough

Reworks the CreateWorkspaceButton UI: replaces HotkeyTooltipContent with a plain "New workspace" tooltip string, and splits the single dropdown into a primary "Create" button plus a separate chevron-triggered DropdownMenu containing the three original actions, with adjusted layout and styling.

Changes

Cohort / File(s) Summary
Create control UI
apps/desktop/src/renderer/screens/main/components/TopBar/WorkspaceTabs/CreateWorkspaceButton.tsx
Replaced HotkeyTooltipContent with a plain tooltip string ("New workspace"); removed previous single DropdownMenuTrigger/DropdownMenuContent composition and introduced a two-part control: a primary "Create" button (immediate action) plus a separate chevron DropdownMenu for secondary options. Preserved menu items (New Workspace, Quick Create, Open Project) and their shortcuts; updated layout classes and icon sizes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • CharlieHelps

Poem

🐰 I nudged a button, split the flow in two,
A primary hop for quick starts, a chevron for view.
Tooltips now simple, no shortcut parade,
I nibble on clarity — a neat UI crusade! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check ⚠️ Warning No pull request description was provided by the author. The template requires at least a Description section with clear and concise details of changes. Add a comprehensive pull request description following the template, including: Description of changes, Related Issues, Type of Change, Testing details, and any applicable Screenshots or Additional Notes.
Title check ❓ Inconclusive The title 'simplify the tooltip' is vague and generic, failing to specify what tooltip is being simplified or what the main change accomplishes. Revise the title to be more specific and descriptive, such as 'Refactor CreateWorkspaceButton tooltip and UI layout' or 'Reorganize CreateWorkspaceButton with primary action and dropdown menu'.
✨ 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 merged commit 5972dc4 into main Jan 1, 2026
5 checks passed
@Kitenite Kitenite deleted the simplify-the-tooltip branch January 1, 2026 19:27
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 1, 2026

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

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

Thank you for your contribution! 🎉

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.

1 participant