Skip to content

fix (desktop): new workspace button layout fix#828

Merged
AviPeltz merged 1 commit intomainfrom
new-workspace-button-shouldnt-be-centered
Jan 19, 2026
Merged

fix (desktop): new workspace button layout fix#828
AviPeltz merged 1 commit intomainfrom
new-workspace-button-shouldnt-be-centered

Conversation

@AviPeltz
Copy link
Copy Markdown
Collaborator

@AviPeltz AviPeltz commented Jan 19, 2026

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
    • Improved the "New Workspace" button layout by adjusting its width to span full container space and realigning the label text for better visual consistency.

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 19, 2026

📝 Walkthrough

Walkthrough

The NewWorkspaceButton component receives styling adjustments: the button container now spans full width using the w-full class, and the label text alignment changes to text-left. No functional behavior or event handling is modified.

Changes

Cohort / File(s) Summary
NewWorkspaceButton Styling
apps/desktop/src/renderer/screens/main/components/WorkspaceSidebar/WorkspaceSidebarHeader/NewWorkspaceButton.tsx
Button width adjusted from auto to full width (w-full); label text alignment changed to left (text-left)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A button stretches wide and true,
Text aligned to the left, fresh and new,
No logic shifted, no code rearranged,
Just pixels dancing, slightly changed!

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
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 ⚠️ Warning The PR description is a mostly empty template with only the 'Bug fix' type selected but missing all substantive content sections. Complete the description with details about what styling issue was fixed, link related issues, and explain the testing performed to verify the layout changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: fixing the layout of the new workspace button on the desktop.

✏️ 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.

@AviPeltz AviPeltz merged commit a19dc3c into main Jan 19, 2026
5 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 new-workspace-button-shouldnt-be-centered branch January 19, 2026 23:00
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