Skip to content

Conversation

@madsrasmussen
Copy link
Contributor

This pull request adds support for displaying the "Trashed" state for documents in the UI. The main changes include introducing a new localized label for trashed content and updating the document info view to show a red tag when a document is trashed.

Screenshot 2025-10-21 at 09 48 00

Introduces a new 'Trashed' label and tag for documents in the workspace view. Updates localization to include the 'Trashed' term for improved clarity when displaying trashed documents.
Copilot AI review requested due to automatic review settings October 21, 2025 07:49
@madsrasmussen madsrasmussen added the category/ux User experience label Oct 21, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds visual indication when a document is in the "Trashed" state within the document workspace info view. The implementation displays a red tag labeled "Trashed" when a document has been moved to the recycle bin.

Key Changes:

  • Added "Trashed" localization key to English language file
  • Implemented UI rendering for trashed document state with a danger-styled tag

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/Umbraco.Web.UI.Client/src/packages/documents/documents/workspace/views/info/document-workspace-view-info.element.ts Added case handler for TRASHED state to render red danger tag
src/Umbraco.Web.UI.Client/src/assets/lang/en.ts Added 'trashed' localization key

@madsrasmussen madsrasmussen changed the title Document Recycle Bin: Add 'Trashed' state to document workspace view Document Recycle Bin: Add 'Trashed' state to document info workspace view Oct 21, 2025
@madsrasmussen madsrasmussen changed the title Document Recycle Bin: Add 'Trashed' state to document info workspace view Document/Media Recycle Bin: Add 'Trashed' state to info workspace view Oct 21, 2025
Copy link
Contributor

@AndyButland AndyButland left a comment

Choose a reason for hiding this comment

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

The updates in this PR look good @madsrasmussen - I can see the "Trashed" state appearing as expected on the Info panel.

I found a couple of little issues though. Will approve as I don't think they are related, but you may want to look at them in this PR or another.

  1. If I view a document in the recycle bin and refresh the browser, I see this error pop-up.
image
  1. When restoring the location to restore too isn't displayed:
image

@madsrasmussen
Copy link
Contributor Author

thanks @AndyButland

Both issues have already been reported 👍

#20515
#20520

@madsrasmussen madsrasmussen merged commit f0ac7b4 into release/17.0 Oct 21, 2025
23 checks passed
@madsrasmussen madsrasmussen deleted the v17/hotfix/show-trashed-state-for-documents branch October 21, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants