From 609607f68a597457ed4a0bb5f6953b9a3f521cd3 Mon Sep 17 00:00:00 2001
From: rabbitism <dongbin@irihi.tech>
Date: Sun, 14 Sep 2025 21:47:31 +0800
Subject: [PATCH] feat: replace ItemsControl with MultiComboBoxSelectedItemList
 and add Remove command for selected items

---
 .../Controls/MultiAutoCompleteBox.axaml       | 20 ++++++-------------
 .../AutoCompleteBox/MultiAutoCompleteBox.cs   |  9 +++++++++
 2 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/src/Ursa.Themes.Semi/Controls/MultiAutoCompleteBox.axaml b/src/Ursa.Themes.Semi/Controls/MultiAutoCompleteBox.axaml
index bb5faa0..0e7a22b 100644
--- a/src/Ursa.Themes.Semi/Controls/MultiAutoCompleteBox.axaml
+++ b/src/Ursa.Themes.Semi/Controls/MultiAutoCompleteBox.axaml
@@ -17,21 +17,13 @@
                         CornerRadius="{TemplateBinding CornerRadius}"
                         Background="{DynamicResource TextBoxDefaultBackground}"
                         BorderBrush="{DynamicResource TextBoxDefaultBorderBrush}">
-                        <ItemsControl 
+                        <u:MultiComboBoxSelectedItemList 
                             Name="{x:Static u:MultiAutoCompleteBox.PART_SelectedItemsControl}"
                             VerticalAlignment="Center"
+                            RemoveCommand="{Binding RelativeSource={RelativeSource TemplatedParent}, Path=Remove}"
+                            ItemTemplate="{TemplateBinding SelectedItemTemplate}"
                             ItemsSource="{TemplateBinding SelectedItems}" >
-                            <ItemsControl.ItemTemplate>
-                                <DataTemplate>
-                                    <!-- TODO: Update command.  -->
-                                    <u:ClosableTag 
-                                        Content="{Binding .}" 
-                                        Command="{Binding $parent[u:TagInput].Close}"
-                                        ContentTemplate="{Binding $parent[u:MultiAutoCompleteBox].SelectedItemTemplate}" 
-                                        ></u:ClosableTag>
-                                </DataTemplate>
-                            </ItemsControl.ItemTemplate>
-                            <ItemsControl.ItemsPanel>
+                            <u:MultiComboBoxSelectedItemList.ItemsPanel>
                                 <ItemsPanelTemplate>
                                     <u:WrapPanelWithTrailingItem>
                                         <u:WrapPanelWithTrailingItem.TrailingItem>
@@ -39,8 +31,8 @@
                                         </u:WrapPanelWithTrailingItem.TrailingItem>
                                     </u:WrapPanelWithTrailingItem>
                                 </ItemsPanelTemplate>
-                            </ItemsControl.ItemsPanel>
-                        </ItemsControl>
+                            </u:MultiComboBoxSelectedItemList.ItemsPanel>
+                        </u:MultiComboBoxSelectedItemList>
                     </Border>
                     <Popup
                         Name="PART_Popup"
diff --git a/src/Ursa/Controls/AutoCompleteBox/MultiAutoCompleteBox.cs b/src/Ursa/Controls/AutoCompleteBox/MultiAutoCompleteBox.cs
index bf72a19..06ee57a 100644
--- a/src/Ursa/Controls/AutoCompleteBox/MultiAutoCompleteBox.cs
+++ b/src/Ursa/Controls/AutoCompleteBox/MultiAutoCompleteBox.cs
@@ -1879,6 +1879,15 @@ public partial class MultiAutoCompleteBox : TemplatedControl, IInnerContentContr
         }
     }
 
+    public void Remove(object? o)
+    {
+        if (o is not StyledElement s) return;
+        if (SelectedItems?.Contains(s.DataContext) == true)
+        {
+            SelectedItems.Remove(s.DataContext);
+        }
+    }
+
     // TODO12: Remove, this shouldn't be part of the public API. Use our internal BindingEvaluator instead.
     /// <summary>
     ///     A framework element that permits a binding to be evaluated in a new data