chore(desktop): improve open project button clarity

[AviPeltz]

Description

Related Issues

Type of Change

• Bug fix
• New feature
• Documentation
• Refactor
• Other (please describe):

Testing

Screenshots (if applicable)

Additional Notes

Summary by CodeRabbit

• Style
  • Updated the action button icon in the workspace creation modal dialog interface.
  • Changed the primary button text label that users interact with.
  • Updated the tooltip message displayed when hovering over the action button.
  • All existing functionality, control flow, and interaction behavior preserved without modification.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

The NewWorkspaceModal component's action button was updated with a new icon (HiFolderOpen replacing HiPlus) and revised button text from "Import" to "Open" with corresponding tooltip text change from "Add new project" to "Open a new project". No control flow or functional logic changes were introduced.

Changes

Cohort / File(s)	Summary
UI Icon & Text Updates in NewWorkspaceModal apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx	Swapped action button icon from HiPlus to HiFolderOpen; updated button text to "Open" and tooltip text to "Open a new project"; existing event handler and disabled state remain unchanged

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• Add dropdown create modal #391: Adds/overhauls the NewWorkspaceModal component; the main PR updates the same button icon and text within it.
• fix(desktop): add tooltip to NewWorkspace modal new project button & make sure default selected #461: Modifies NewWorkspaceModal's action button logic and adds Tooltip wrapper; the main PR further refines the button's icon and text presentation.
• ui(desktop): new button with shortcuts option #460: Refactors/expands the handleOpenNewProject flow that this PR's button still invokes.

Suggested reviewers

• CharlieHelps

Poem

🐰 A button transformed with folded grace,
From plus sign pride to folder's face,
"Import" whispers, now "Open" calls,
Same action flows through UI halls,
A cosmetic bloom, small but bright! 📁✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is entirely empty—all template sections are unpopulated with no actual content provided beyond the template structure.	Complete the description with: a summary of changes (icon swap, text updates), related issues if any, mark 'Other' for type of change, describe testing performed, and add any relevant context.
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.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main UI change: improving button clarity by replacing the icon and updating text from 'Import' to 'Open' on the project button.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch new-project-button

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f075b17 and 3c1fb6b.

📒 Files selected for processing (1)

• apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx

🧰 Additional context used

📓 Path-based instructions (5)

apps/desktop/**/*.{ts,tsx}

📄 CodeRabbit inference engine (apps/desktop/AGENTS.md)

apps/desktop/**/*.{ts,tsx}: For Electron interprocess communication, ALWAYS use tRPC as defined in src/lib/trpc
Use alias as defined in tsconfig.json when possible
Prefer zustand for state management if it makes sense. Do not use effect unless absolutely necessary.
For tRPC subscriptions with trpc-electron, ALWAYS use the observable pattern from @trpc/server/observable instead of async generators, as the library explicitly checks isObservable(result) and throws an error otherwise

Files:

• apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx

**/*.{ts,tsx}

📄 CodeRabbit inference engine (AGENTS.md)

Use type safety and avoid any types unless absolutely necessary in TypeScript files

Files:

• apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx

apps/**/src/**/**/[A-Z]*.tsx

📄 CodeRabbit inference engine (AGENTS.md)

Structure component folders with one component per file using format ComponentName/ComponentName.tsx with index.ts barrel export

Files:

• apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx

apps/desktop/src/**/*.{ts,tsx}

📄 CodeRabbit inference engine (AGENTS.md)

Move Node.js functionality needed in Electron renderer to src/main/lib/ and communicate via IPC

Files:

• apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx

apps/desktop/src/{shared/ipc-channels.ts,main/**/*ipcs.ts,renderer/**/*.tsx}

📄 CodeRabbit inference engine (AGENTS.md)

Define all Electron IPC channel types in apps/desktop/src/shared/ipc-channels.ts before implementing handlers

Files:

• apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx

🧬 Code graph analysis (1)

apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx (2)

packages/ui/src/components/ui/button.tsx (1)

• Button (60-60)

packages/ui/src/components/ui/tooltip.tsx (2)

• TooltipTrigger (76-76)
• TooltipContent (76-76)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build

🔇 Additional comments (2)

apps/desktop/src/renderer/components/NewWorkspaceModal/NewWorkspaceModal.tsx (2)

37-37: LGTM! Icon import improves semantic clarity.

The HiFolderOpen icon is more semantically appropriate for opening/browsing projects than the generic HiPlus icon that was previously used.

---

267-272: LGTM! UI changes improve clarity and consistency.

The updated icon, button text, and tooltip work together to clearly communicate the action. The terminology is now consistent with the dialog title ("Open Workspace") and better describes the user flow compared to the previous "Import"/"Add" wording.

---

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 and usage tips.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

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

Thank you for your contribution! 🎉