Skip to content

Conversation

@nielslyngsoe
Copy link
Member

Added a new icon and used it for deleting things in the trash bin

@Copilot Copilot AI review requested due to automatic review settings October 23, 2025 11:41
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 introduces a new icon-trash-empty icon to represent delete actions in the trash bin/recycle bin. The icon is sourced from the updated lucide-static library and applied to delete entity actions across documents and media.

  • Adds new icon-trash-empty icon from lucide-static
  • Updates delete actions for documents and media to use the new icon
  • Updates empty recycle bin action to use the new icon

Reviewed Changes

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

Show a summary per file
File Description
src/Umbraco.Web.UI.Client/src/packages/core/package.json Updates lucide-static dependency from 0.544.0 to 0.546.0
src/Umbraco.Web.UI.Client/src/packages/core/icon-registry/icons/icon-trash-empty.ts Adds new trash-empty icon SVG definition
src/Umbraco.Web.UI.Client/src/packages/core/icon-registry/icons.ts Registers the new icon in the icon registry
src/Umbraco.Web.UI.Client/src/packages/core/icon-registry/icon-dictionary.json Adds icon metadata to the dictionary
src/Umbraco.Web.UI.Client/src/packages/core/recycle-bin/entity-action/empty-recycle-bin/empty-recycle-bin.action.kind.ts Updates empty recycle bin action to use new icon
src/Umbraco.Web.UI.Client/src/packages/documents/documents/entity-actions/manifests.ts Applies new icon to document delete action
src/Umbraco.Web.UI.Client/src/packages/media/media/entity-actions/manifests.ts Applies new icon to media delete action
src/Umbraco.Web.UI.Client/src/packages/core/icon-registry/icons/icon-company.ts Updates company icon SVG (unrelated icon update from lucide-static upgrade)
Files not reviewed (1)
  • src/Umbraco.Web.UI.Client/package-lock.json: Language not supported

@nielslyngsoe nielslyngsoe enabled auto-merge (squash) October 23, 2025 11:41
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.

1 participant