From 9ab31b037e1ea14dc6e8a1bfbe73f285a172e94e Mon Sep 17 00:00:00 2001
From: Lester Caine <lester@lsces.co.uk>
Date: Fri, 5 Jun 2026 09:35:18 +0100
Subject: Switch delete icon from edit-delete to user-trash (dustbin)

user-trash updated to a proper dustbin SVG. All templates across
all packages updated to use user-trash instead of edit-delete.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 templates/admin_xref_groups.tpl            | 2 +-
 templates/admin_xref_sources.tpl           | 2 +-
 templates/content_permissions_inc.tpl      | 2 +-
 templates/content_role_permissions_inc.tpl | 2 +-
 templates/display_comment.tpl              | 2 +-
 templates/edit_storage_list.tpl            | 4 ++--
 templates/structure_toc_leaf.tpl           | 2 +-
 templates/view_xref_address_item.tpl       | 2 +-
 templates/view_xref_bank_item.tpl          | 2 +-
 templates/view_xref_contact_item.tpl       | 2 +-
 templates/view_xref_date_item.tpl          | 2 +-
 templates/view_xref_locate_item.tpl        | 2 +-
 templates/view_xref_phone_item.tpl         | 2 +-
 templates/view_xref_sig_item.tpl           | 2 +-
 templates/view_xref_text_item.tpl          | 2 +-
 templates/view_xref_value_item.tpl         | 2 +-
 16 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/templates/admin_xref_groups.tpl b/templates/admin_xref_groups.tpl
index fd0c1d9..212428a 100644
--- a/templates/admin_xref_groups.tpl
+++ b/templates/admin_xref_groups.tpl
@@ -85,7 +85,7 @@
 					<td>
 						{if $grp.num_sources eq 0}
 							<a href="{$smarty.const.LIBERTY_PKG_URL}admin/admin_xref_groups.php?fDeleteGroup=1&amp;x_group={$grp.x_group|escape}&amp;del_content_type_guid={$grp.content_type_guid|escape}&amp;content_type_guid={$activeGuid|escape}"
-							   onclick="return confirm('{tr}Delete this group?{/tr}')">{biticon ipackage="icons" iname="edit-delete" ipackage="icons" iforce=icon_text iexplain="Delete"}</a>
+							   onclick="return confirm('{tr}Delete this group?{/tr}')">{biticon ipackage="icons" iname="user-trash" ipackage="icons" iforce=icon_text iexplain="Delete"}</a>
 
 						{/if}
 					</td>
diff --git a/templates/admin_xref_sources.tpl b/templates/admin_xref_sources.tpl
index 594c1d6..8812772 100644
--- a/templates/admin_xref_sources.tpl
+++ b/templates/admin_xref_sources.tpl
@@ -105,7 +105,7 @@
 					<td>
 						{if $src.num_entries eq 0}
 							<a href="{$smarty.const.LIBERTY_PKG_URL}admin/admin_xref_sources.php?fDeleteSource=1&amp;source={$src.item|escape}&amp;del_content_type_guid={$src.content_type_guid|escape}&amp;content_type_guid={$activeGuid|escape}"
-							   onclick="return confirm('{tr}Delete this source?{/tr}')">{biticon ipackage="icons" iname="edit-delete" ipackage="icons" iforce=icon_text iexplain="Delete"}</a>
+							   onclick="return confirm('{tr}Delete this source?{/tr}')">{biticon ipackage="icons" iname="user-trash" ipackage="icons" iforce=icon_text iexplain="Delete"}</a>
 						{/if}
 					</td>
 				</tr>
diff --git a/templates/content_permissions_inc.tpl b/templates/content_permissions_inc.tpl
index 940e20d..e4c78e5 100644
--- a/templates/content_permissions_inc.tpl
+++ b/templates/content_permissions_inc.tpl
@@ -83,7 +83,7 @@
 					<td>{$perm.group_name}</td>
 					<td>{$perm.perm_desc} <em>({$perm.perm_name})</em></td>
 					<td class="alignright">
-						{smartlink ititle="Remove Permission" biticon="edit-delete" action=remove content_id=$gContent->mContentId perm=$perm.perm_name group_id=$perm.group_id}
+						{smartlink ititle="Remove Permission" biticon="user-trash" action=remove content_id=$gContent->mContentId perm=$perm.perm_name group_id=$perm.group_id}
 					</td>
 				</tr>
 			{foreachelse}
diff --git a/templates/content_role_permissions_inc.tpl b/templates/content_role_permissions_inc.tpl
index 136d87b..f1a64a8 100644
--- a/templates/content_role_permissions_inc.tpl
+++ b/templates/content_role_permissions_inc.tpl
@@ -83,7 +83,7 @@
 					<td>{$perm.role_name}</td>
 					<td>{$perm.perm_desc} <em>({$perm.perm_name})</em></td>
 					<td class="alignright">
-						{smartlink ititle="Remove Permission" biticon="edit-delete" action=remove content_id=$gContent->mContentId perm=$perm.perm_name role_id=$perm.role_id}
+						{smartlink ititle="Remove Permission" biticon="user-trash" action=remove content_id=$gContent->mContentId perm=$perm.perm_name role_id=$perm.role_id}
 					</td>
 				</tr>
 			{foreachelse}
diff --git a/templates/display_comment.tpl b/templates/display_comment.tpl
index e431d63..3f2c3e9 100755
--- a/templates/display_comment.tpl
+++ b/templates/display_comment.tpl
@@ -17,7 +17,7 @@
 				<a href="{$comments_return_url}&amp;post_comment_id={$comment.comment_id}&amp;post_comment_request=1#editcomments" rel="nofollow">{biticon ipackage="icons" iname="document-properties" ipackage="icons" iexplain="Edit"}</a>
 			{/if}
 			{if $gBitUser->hasPermission('p_liberty_admin_comments')}
-				<a href="{$comments_return_url}&amp;delete_comment_id={$comment.comment_id}" rel="nofollow">{biticon ipackage="icons" iname="edit-delete" iexplain="Remove"}</a>
+				<a href="{$comments_return_url}&amp;delete_comment_id={$comment.comment_id}" rel="nofollow">{biticon ipackage="icons" iname="user-trash" iexplain="Remove"}</a>
 			{/if}
 		</div>
 
diff --git a/templates/edit_storage_list.tpl b/templates/edit_storage_list.tpl
index d48485a..6b8b9ef 100755
--- a/templates/edit_storage_list.tpl
+++ b/templates/edit_storage_list.tpl
@@ -70,13 +70,13 @@ window.attachmentUrls["{$storage.attachment_id}"]["original"] = "{$storage.sourc
 								<a href="javascript:void(0);" onclick="
 									BitAjax.updater('edit_storage_list_tab_{$form_id}', '{$smarty.const.LIBERTY_PKG_URL}ajax_edit_storage.php', '{$smarty.capture.urlArgs}&amp;deleteAttachment={$attachmentId}&amp;form_id={$form_id}');
 									BitAjax.updater('edit_storage_list_{$form_id}', '{$smarty.const.LIBERTY_PKG_URL}ajax_edit_storage.php', '{$smarty.capture.urlArgs}&amp;form_id={$form_id}');">
-										{biticon ipackage="icons" iname="edit-delete" iexplain="delete"}
+										{biticon ipackage="icons" iname="user-trash" iexplain="delete"}
 								</a>
 							{else}
 								{if $storage.is_mime}
 									<a href="{$storage.display_url}">{biticon ipackage="icons" iname="folder-open" iexplain="View"}</a>
 								{/if}
-								<a href="{$smarty.server.SCRIPT_NAME}?{$smarty.capture.urlArgs}&amp;deleteAttachment={$attachmentId}">{biticon ipackage="icons" iname="edit-delete" iexplain="Delete"}</a>
+								<a href="{$smarty.server.SCRIPT_NAME}?{$smarty.capture.urlArgs}&amp;deleteAttachment={$attachmentId}">{biticon ipackage="icons" iname="user-trash" iexplain="Delete"}</a>
 							{/if}
 						{/if}
 					</div>
diff --git a/templates/structure_toc_leaf.tpl b/templates/structure_toc_leaf.tpl
index f3b063d..6c2f4c8 100755
--- a/templates/structure_toc_leaf.tpl
+++ b/templates/structure_toc_leaf.tpl
@@ -2,7 +2,7 @@
 <li {if $editingStructure}id="structure-node-{$structure_tree.structure_id}" content_id="{$structure_tree.content_id}" structure_id="{$structure_tree.structure_id}"{/if} {if $structure_tree.structure_id==$smarty.request.structure_id}class="highlight"{/if}>
 		{if $editingStructure}
 		<div class="inline-block">
-			<div class="btn btn-default btn-xs" onclick="deleteStructureNode('{$structure_tree.structure_id}','{$structure_tree.title|escape:javascript}')">{biticon ipackage="icons" iname="edit-delete" class="icon"}</div>
+			<div class="btn btn-default btn-xs" onclick="deleteStructureNode('{$structure_tree.structure_id}','{$structure_tree.title|escape:javascript}')">{biticon ipackage="icons" iname="user-trash" class="icon"}</div>
 			<span class="inline-block" style="padding:5px;">{biticon ipackage="icons" iname="go-next" class="structure-sort-handle"}</span>
 		</div>
 		{/if}
diff --git a/templates/view_xref_address_item.tpl b/templates/view_xref_address_item.tpl
index 965430f..cdccba7 100644
--- a/templates/view_xref_address_item.tpl
+++ b/templates/view_xref_address_item.tpl
@@ -20,7 +20,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_bank_item.tpl b/templates/view_xref_bank_item.tpl
index f26fc1f..af0d9fb 100644
--- a/templates/view_xref_bank_item.tpl
+++ b/templates/view_xref_bank_item.tpl
@@ -20,7 +20,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_contact_item.tpl b/templates/view_xref_contact_item.tpl
index ad4ee11..72ea90e 100644
--- a/templates/view_xref_contact_item.tpl
+++ b/templates/view_xref_contact_item.tpl
@@ -26,7 +26,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_date_item.tpl b/templates/view_xref_date_item.tpl
index f26fc1f..af0d9fb 100644
--- a/templates/view_xref_date_item.tpl
+++ b/templates/view_xref_date_item.tpl
@@ -20,7 +20,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_locate_item.tpl b/templates/view_xref_locate_item.tpl
index f26fc1f..af0d9fb 100644
--- a/templates/view_xref_locate_item.tpl
+++ b/templates/view_xref_locate_item.tpl
@@ -20,7 +20,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_phone_item.tpl b/templates/view_xref_phone_item.tpl
index 344e511..7eeedfe 100644
--- a/templates/view_xref_phone_item.tpl
+++ b/templates/view_xref_phone_item.tpl
@@ -20,7 +20,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_sig_item.tpl b/templates/view_xref_sig_item.tpl
index f26fc1f..af0d9fb 100644
--- a/templates/view_xref_sig_item.tpl
+++ b/templates/view_xref_sig_item.tpl
@@ -20,7 +20,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_text_item.tpl b/templates/view_xref_text_item.tpl
index ad4ee11..72ea90e 100644
--- a/templates/view_xref_text_item.tpl
+++ b/templates/view_xref_text_item.tpl
@@ -26,7 +26,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
diff --git a/templates/view_xref_value_item.tpl b/templates/view_xref_value_item.tpl
index 96ac916..9322b7b 100644
--- a/templates/view_xref_value_item.tpl
+++ b/templates/view_xref_value_item.tpl
@@ -20,7 +20,7 @@
 			{if $source eq 'history'}
 				{smartlink ititle="Restore" ipackage="liberty" ifile="edit_xref.php" biticon="document-properties" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=-1}
 			{else}
-				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="edit-delete" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
+				{smartlink ititle="Delete" ipackage="liberty" ifile="edit_xref.php" biticon="user-trash" content_id=$gContent->mInfo.content_id xref_id=$gContent->mInfo.$source[xref].xref_id expunge=1}
 			{/if}
 		{/if}
 	</span>
-- 
cgit v1.3