Skip to content

refactor(web): remove redundant add-project cancel button#1302

Merged
juliusmarminge merged 1 commit intopingdotgg:mainfrom
UtkarshUsername:fix/remove-add-project-cancel-button
Mar 30, 2026
Merged

refactor(web): remove redundant add-project cancel button#1302
juliusmarminge merged 1 commit intopingdotgg:mainfrom
UtkarshUsername:fix/remove-add-project-cancel-button

Conversation

@UtkarshUsername
Copy link
Copy Markdown
Contributor

@UtkarshUsername UtkarshUsername commented Mar 22, 2026

What Changed

Removed the extra inline Cancel button from the add-project form in the sidebar.

Why

There is already another cancel button. The add-project toggle button switches into the cancel action when the form is open.
The inline Cancel button duplicates behavior that is already exposed by this add-project toggle. (You can also press Esc to cancel btw)
Removing it simplifies the UI and reduces duplicate controls for the same action.

UI Changes

Before:
image

After:
image

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes

Note

Remove Cancel button from add-project panel in Sidebar

Removes the inline Cancel button from the add-project UI in Sidebar.tsx. Risk: users can no longer dismiss the add-project panel via this button.

Macroscope summarized fc67e4d.


Note

Low Risk
Low risk UI simplification in the sidebar add-project form; main behavioral change is removing one way to cancel/clear errors, relying on the existing toggle button and Esc instead.

Overview
Simplifies the sidebar add-project UI by removing the inline Cancel button from the add-project form in Sidebar.tsx.

Canceling add-project mode (and clearing addProjectError) is now done via the existing add-project toggle button (plus icon) or the Escape key, reducing duplicated controls.

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

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 22, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 942e06b0-102d-4b63-9e00-265fd76e795f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions github-actions bot added size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Mar 22, 2026
@UtkarshUsername UtkarshUsername force-pushed the fix/remove-add-project-cancel-button branch from dbaa4a2 to fc67e4d Compare March 30, 2026 11:16
@juliusmarminge juliusmarminge merged commit 6f51725 into pingdotgg:main Mar 30, 2026
11 checks passed
aaditagrawal added a commit to aaditagrawal/t3code that referenced this pull request Mar 30, 2026
Integrates upstream changes including:

- Refactor terminal manager onto Effect runtime (pingdotgg#1525)
- Refactor web orchestration sync to incremental events and
  isolated recovery (pingdotgg#1560)
- Remove redundant add-project cancel button (pingdotgg#1302)
- README documentation updates (pingdotgg#1406, pingdotgg#1564, pingdotgg#1565)

Conflict resolution across 8 files: adopted upstream's incremental
event store architecture, terminal Effect runtime, and batched
orchestration effects while preserving fork's multi-provider state.
Chrono-byte pushed a commit to Chrono-byte/t3code that referenced this pull request Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants