Skip to content

update project removal copy#981

Merged
juliusmarminge merged 1 commit intopingdotgg:mainfrom
donnes:fix/sidebar-remove-project-copy
Mar 12, 2026
Merged

update project removal copy#981
juliusmarminge merged 1 commit intopingdotgg:mainfrom
donnes:fix/sidebar-remove-project-copy

Conversation

@donnes
Copy link
Contributor

@donnes donnes commented Mar 12, 2026

What Changed

  • Renamed the sidebar project action from Delete to Remove project.
  • Updated the confirmation and error copy to use remove wording for project removal.
  • Kept the non-empty project warning focused on deleting threads before removing the project.

Why

  • Delete reads as more destructive than the actual sidebar action and can imply the project folder itself is being deleted.
  • Remove project better matches what T3 Code actually does and reduces confusion.

UI Changes

  • Text-only UI copy change in the sidebar project removal flow.

Checklist

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

Note

Update project removal copy in sidebar context menu

Renames the context menu action from 'Delete' to 'Remove project' and updates all related copy in Sidebar.tsx to use 'remove' terminology consistently, including the warning toast, confirmation dialog, and error messages.

Macroscope summarized 783b626.

@coderabbitai
Copy link

coderabbitai bot commented Mar 12, 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: 8ca611fd-a550-4f8f-b000-d4f827903ca6

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
  • Post copyable unit tests in a comment
📝 Coding Plan for PR comments
  • Generate coding plan

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 the vouch:unvouched PR author is not yet trusted in the VOUCHED list. label Mar 12, 2026
@donnes donnes changed the title Update project removal copy update project removal copy Mar 12, 2026
@juliusmarminge juliusmarminge merged commit ab0002f into pingdotgg:main Mar 12, 2026
7 of 8 checks passed
juliusmarminge pushed a commit to hobostay/t3code that referenced this pull request Mar 12, 2026
gabrielMalonso pushed a commit to gabrielMalonso/t3code that referenced this pull request Mar 12, 2026
gabrielMalonso added a commit to gabrielMalonso/t3code that referenced this pull request Mar 12, 2026
* fix: don't restart the app on normal exists in dev runner (pingdotgg#986)

Co-authored-by: Julius Marminge <julius0216@outlook.com>

* feat: add fuzzy workspace entry search (pingdotgg#256)

Co-authored-by: Julius Marminge <julius0216@outlook.com>

* fix: clean up timeout in PlanSidebar to prevent memory leaks (pingdotgg#949)

Co-authored-by: hobostay <hobostay@users.noreply.github.com>

* fix: Linux icon now shows up (pingdotgg#807)

* fix syntax errors from bad merge

* fix: add logging for WebSocket errors (pingdotgg#948)

Co-authored-by: hobostay <hobostay@users.noreply.github.com>

* fix: add error logging for code highlighting failures (pingdotgg#951)

* chore: update actions/checkout and actions/github-script (pingdotgg#956)

Co-authored-by: Julius Marminge <julius0216@outlook.com>

* update project removal copy (pingdotgg#981)

* fix pr size workflow

* Fix new-thread shortcuts when terminal is focused

- move chat-wide key handling into `_chat` route-level shortcut handler
- extract reusable `useHandleNewThread` hook and `isTerminalFocused` helper
- update browser WS fixture to support `terminalOpen` RPC shape

* Fix mod+N new thread flow and terminal split limits

- ensure `chat.new` creates a fresh draft after a promoted draft thread
- enforce terminal cap per split group (4) while allowing additional terminal groups
- refine sidebar row selected/active styling via shared class-name logic and tests

* fix(web): add pointer cursor to running stop-generation button (pingdotgg#900)

Co-authored-by: Julius Marminge <julius0216@outlook.com>

* fix: add missing closing brace in wsTransport connect method

The cherry-pick of WebSocket error logging (b6eba33) left a missing
closing brace for the connect() method, causing a TS compilation error.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* style: fix formatting in _chat.tsx

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Noah Gregory <noah@wts.dev>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Kiyotaka <maskdotdev@gmail.com>
Co-authored-by: Qiaochu Hu <110hqc@gmail.com>
Co-authored-by: hobostay <hobostay@users.noreply.github.com>
Co-authored-by: Alex Schwartz <alexschwartz01@gmail.com>
Co-authored-by: Adam Naji <110662505+Bashamega@users.noreply.github.com>
Co-authored-by: Donald Silveira <donaldsilveira@gmail.com>
Co-authored-by: 0x1f99d <crazywolf132@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants