Skip to content

feat: add open project button to WelcomeScreen#116

Merged
dannysmith merged 3 commits intodannysmith:mainfrom
nickradford:main
Feb 11, 2026
Merged

feat: add open project button to WelcomeScreen#116
dannysmith merged 3 commits intodannysmith:mainfrom
nickradford:main

Conversation

@nickradford
Copy link
Copy Markdown
Contributor

@nickradford nickradford commented Feb 6, 2026 •
edited by coderabbitai bot
Loading

As mentioned in #82 (comment). I saw our timezone differences and wanted to open this in case, feel free to reject if this steps on the proper Projects UI work.

Simply adds an Open Project button to the Welcome Screen to clearly direct users to the intended first action.

image

Summary by CodeRabbit

  • New Features

    • Added an "Open Project" button to the Welcome screen, making it easier to open existing projects via a file dialog.

  • Style

    • Refined Welcome screen layout, spacing, and heading alignment for improved visual organization and clarity.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 6, 2026 •
edited
Loading

Warning

Rate limit exceeded

@nickradford has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 15 minutes and 30 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

MainEditor.tsx updated: added imports for openProjectViaDialog and Button; Welcome screen layout adjusted to flex column with gap spacing and heading margin tweaks; an "Open Project" button was added that calls openProjectViaDialog (promise ignored) on click.

Changes

Cohort / File(s) Summary
Welcome Screen Enhancement
src/components/layout/MainEditor.tsx		 Imported openProjectViaDialog and Button; restructured Welcome screen to use flex + flex-col with gap-4 and adjusted heading margins; added centered "Open Project" Button (variant outline, size="sm") that calls openProjectViaDialog via an ignored-promise wrapper on click.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A tiny paw taps a brand-new door,
A button gleams — "Open Project" once more,
Columns align and gaps fall in line,
I hop, I click, the dialog's mine! 🥕✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding an Open Project button to the WelcomeScreen component. It directly reflects the code modifications shown in the summary.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

coderabbitai[bot]
coderabbitai bot reviewed Feb 6, 2026
View reviewed changes
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents 
In `@src/components/layout/MainEditor.tsx`:
- Line 10: The container div in MainEditor.tsx currently uses both "space-y-4"
and "gap-4" which double-applies vertical spacing; remove "space-y-4" from the
className on the <div className="text-center text-muted-foreground space-y-4
flex flex-col gap-4"> (leave "gap-4") so the flex gap alone controls spacing
between children.

@dannysmith dannysmith merged commit d55808b into dannysmith:main Feb 11, 2026
4 checks passed
@dannysmith
Copy link
Copy Markdown
Owner

dannysmith commented Feb 11, 2026

@nickradford Thanks for this mate! Just merged and will ship in the next patch version along with a bunch of dependency updates and one or two other little bits.

PS. Thanks for keeping the PR super-small/focussed ❤️

@nickradford
Copy link
Copy Markdown
Contributor Author

nickradford commented Feb 12, 2026

@dannysmith Of course! As you move on to working on #82 if you want an extra set of hands on anything I'd be happy to help. I've got loads of free time right now.

Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nickradford @dannysmith