From c0c2fde7636079c5622179be356f8b3eaa762a82 Mon Sep 17 00:00:00 2001
From: Kelv <kelv@discourse.org>
Date: Thu, 9 Jan 2025 17:46:40 +0800
Subject: [PATCH] DEV: update deprecated icon name from user-times to
 user-xmark (#624)

---
 .../discourse/components/assign-actions-dropdown.js  |  4 ++--
 .../discourse/components/assign-button.gjs           |  2 +-
 .../discourse/initializers/extend-for-assigns.js     | 12 ++++++------
 plugin.rb                                            |  2 +-
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/assets/javascripts/discourse/components/assign-actions-dropdown.js b/assets/javascripts/discourse/components/assign-actions-dropdown.js
index 1f21a7af..2df6f5e8 100644
--- a/assets/javascripts/discourse/components/assign-actions-dropdown.js
+++ b/assets/javascripts/discourse/components/assign-actions-dropdown.js
@@ -21,7 +21,7 @@ export default class AssignActionsDropdown extends DropdownSelectBoxComponent {
       options = options.concat([
         {
           id: "unassign",
-          icon: this.group ? "group-times" : "user-times",
+          icon: this.group ? "group-times" : "user-xmark",
           name: I18n.t("discourse_assign.unassign.title"),
           description: I18n.t("discourse_assign.unassign.help", {
             username: this.assignee,
@@ -42,7 +42,7 @@ export default class AssignActionsDropdown extends DropdownSelectBoxComponent {
         const assignee = assignment_map.assigned_to;
         options = options.concat({
           id: `unassign_post_${postId}`,
-          icon: assignee.username ? "user-times" : "group-times",
+          icon: assignee.username ? "user-xmark" : "group-times",
           name: I18n.t("discourse_assign.unassign_post.title"),
           description: I18n.t("discourse_assign.unassign_post.help", {
             username: assignee.username || assignee.name,
diff --git a/assets/javascripts/discourse/components/assign-button.gjs b/assets/javascripts/discourse/components/assign-button.gjs
index 8e8cb5c0..3a54cc4f 100644
--- a/assets/javascripts/discourse/components/assign-button.gjs
+++ b/assets/javascripts/discourse/components/assign-button.gjs
@@ -15,7 +15,7 @@ export default class AssignButton extends Component {
   @service taskActions;
 
   get icon() {
-    return this.isAssigned ? "user-times" : "user-plus";
+    return this.isAssigned ? "user-xmark" : "user-plus";
   }
 
   get isAssigned() {
diff --git a/assets/javascripts/discourse/initializers/extend-for-assigns.js b/assets/javascripts/discourse/initializers/extend-for-assigns.js
index 999ea32a..3dce8f09 100644
--- a/assets/javascripts/discourse/initializers/extend-for-assigns.js
+++ b/assets/javascripts/discourse/initializers/extend-for-assigns.js
@@ -51,7 +51,7 @@ function registerTopicFooterButtons(api) {
     icon() {
       return this.topic.isAssigned()
         ? this.site.mobileView
-          ? "user-times"
+          ? "user-xmark"
           : null
         : "user-plus";
     },
@@ -150,7 +150,7 @@ function registerTopicFooterButtons(api) {
   api.registerTopicFooterButton({
     id: "unassign-mobile",
     icon() {
-      return "user-times";
+      return "user-xmark";
     },
     translatedTitle() {
       return defaultTitle(this.topic);
@@ -398,9 +398,9 @@ function initialize(api) {
   api.addPostSmallActionIcon("assigned_to_post", "user-plus");
   api.addPostSmallActionIcon("assigned_group", "group-plus");
   api.addPostSmallActionIcon("assigned_group_to_post", "group-plus");
-  api.addPostSmallActionIcon("unassigned", "user-times");
+  api.addPostSmallActionIcon("unassigned", "user-xmark");
   api.addPostSmallActionIcon("unassigned_group", "group-times");
-  api.addPostSmallActionIcon("unassigned_from_post", "user-times");
+  api.addPostSmallActionIcon("unassigned_from_post", "user-xmark");
   api.addPostSmallActionIcon("unassigned_group_from_post", "group-times");
   api.includePostAttributes("assigned_to_user", "assigned_to_group");
   api.addPostSmallActionIcon("reassigned", "user-plus");
@@ -777,7 +777,7 @@ function customizeWidgetPostMenu(api) {
     if (post.assigned_to_user || post.assigned_to_group) {
       return {
         action: "unassignPost",
-        icon: "user-times",
+        icon: "user-xmark",
         className: "unassign-post",
         title: "discourse_assign.unassign_post.title",
         position:
@@ -874,7 +874,7 @@ export default {
       api.addBulkActionButton({
         id: "unassign-topics",
         label: "topics.bulk.unassign",
-        icon: "user-times",
+        icon: "user-xmark",
         class: "btn-default unassign-topics",
         action({ performAndRefresh }) {
           performAndRefresh({ type: "unassign" });
diff --git a/plugin.rb b/plugin.rb
index a29fe3af..18620960 100644
--- a/plugin.rb
+++ b/plugin.rb
@@ -12,7 +12,7 @@
 register_asset "stylesheets/assigns.scss"
 register_asset "stylesheets/mobile/assigns.scss", :mobile
 
-%w[user-plus user-times group-plus group-times].each { |i| register_svg_icon(i) }
+%w[user-plus user-xmark group-plus group-times].each { |i| register_svg_icon(i) }
 
 module ::DiscourseAssign
   PLUGIN_NAME = "discourse-assign"