From 862d4157d289ef378a3d0b31feae99cdce7b7c2a Mon Sep 17 00:00:00 2001
From: Dong Bin <popmessiah@hotmail.com>
Date: Tue, 25 Feb 2025 01:05:06 +0800
Subject: [PATCH 1/4] feat: fix demo name. add a failing case.

---
 ...{SelectionBoxDemo.axaml => SelectionListDemo.axaml} | 10 +++++++---
 ...tionBoxDemo.axaml.cs => SelectionListDemo.axaml.cs} |  0
 2 files changed, 7 insertions(+), 3 deletions(-)
 rename demo/Ursa.Demo/Pages/{SelectionBoxDemo.axaml => SelectionListDemo.axaml} (89%)
 rename demo/Ursa.Demo/Pages/{SelectionBoxDemo.axaml.cs => SelectionListDemo.axaml.cs} (100%)

diff --git a/demo/Ursa.Demo/Pages/SelectionBoxDemo.axaml b/demo/Ursa.Demo/Pages/SelectionListDemo.axaml
similarity index 89%
rename from demo/Ursa.Demo/Pages/SelectionBoxDemo.axaml
rename to demo/Ursa.Demo/Pages/SelectionListDemo.axaml
index 18068cf..ec4a9c3 100644
--- a/demo/Ursa.Demo/Pages/SelectionBoxDemo.axaml
+++ b/demo/Ursa.Demo/Pages/SelectionListDemo.axaml
@@ -12,10 +12,14 @@
     x:DataType="vm:SelectionListDemoViewModel"
     mc:Ignorable="d">
     <StackPanel HorizontalAlignment="Left">
-        <u:SelectionList ItemsSource="{Binding Items}" SelectedItem="{Binding SelectedItem}">
+        <u:SelectionList
+            Width="600"
+            HorizontalAlignment="Center"
+            ItemsSource="{Binding Items}"
+            SelectedItem="{Binding SelectedItem}">
             <u:SelectionList.ItemsPanel>
                 <ItemsPanelTemplate>
-                    <StackPanel Orientation="Horizontal" />
+                    <StackPanel HorizontalAlignment="Center" Orientation="Horizontal" />
                 </ItemsPanelTemplate>
             </u:SelectionList.ItemsPanel>
         </u:SelectionList>
@@ -36,9 +40,9 @@
                 <DataTemplate>
                     <Panel Height="40">
                         <TextBlock
-                            Classes.Active="{Binding $parent[u:SelectionListItem].IsSelected, Mode=OneWay}"
                             Margin="8,0"
                             VerticalAlignment="Center"
+                            Classes.Active="{Binding $parent[u:SelectionListItem].IsSelected, Mode=OneWay}"
                             Text="{Binding}">
                             <TextBlock.Styles>
                                 <Style Selector="TextBlock.Active">
diff --git a/demo/Ursa.Demo/Pages/SelectionBoxDemo.axaml.cs b/demo/Ursa.Demo/Pages/SelectionListDemo.axaml.cs
similarity index 100%
rename from demo/Ursa.Demo/Pages/SelectionBoxDemo.axaml.cs
rename to demo/Ursa.Demo/Pages/SelectionListDemo.axaml.cs

From 3b129b11288614f836c0284b5fcb2d9bfe37a4a9 Mon Sep 17 00:00:00 2001
From: Dong Bin <popmessiah@hotmail.com>
Date: Tue, 25 Feb 2025 01:22:24 +0800
Subject: [PATCH 2/4] fix: translate to panel root.

---
 src/Ursa/Controls/SelectionList/SelectionList.cs | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/src/Ursa/Controls/SelectionList/SelectionList.cs b/src/Ursa/Controls/SelectionList/SelectionList.cs
index 1226032..cbce4ea 100644
--- a/src/Ursa/Controls/SelectionList/SelectionList.cs
+++ b/src/Ursa/Controls/SelectionList/SelectionList.cs
@@ -58,7 +58,7 @@ public class SelectionList: SelectingItemsControl
         }
         OpacityProperty.SetValue(1d, _indicator);
         InvalidateMeasure();
-        InvalidateArrange();
+        // InvalidateArrange();
     }
 
     protected override Size ArrangeOverride(Size finalSize)
@@ -68,7 +68,12 @@ public class SelectionList: SelectingItemsControl
         {
             var container = ContainerFromItem(SelectedItem);
             if (container is null) return size;
-            _indicator.Arrange(container.Bounds);
+            var bounds = container.Bounds;
+            if (ItemsPanelRoot?.Bounds.Position is { } p)
+            {
+                bounds = bounds.Translate(p);
+            }
+            _indicator.Arrange(bounds);
         }
         else
         {
@@ -103,7 +108,12 @@ public class SelectionList: SelectingItemsControl
             var container = ContainerFromItem(SelectedItem);
             if (container is null) return;
             _indicator.Opacity = 1;
-            _indicator.Arrange(container.Bounds);
+            var bounds = container.Bounds;
+            if (ItemsPanelRoot?.Bounds.Position is { } p)
+            {
+                bounds = bounds.Translate(p);
+            }
+            _indicator.Arrange(bounds);
         }
     }
 

From 27146acc4990e7c29152dd0fdb30d59ab8d250d7 Mon Sep 17 00:00:00 2001
From: Dong Bin <popmessiah@hotmail.com>
Date: Tue, 25 Feb 2025 01:31:59 +0800
Subject: [PATCH 3/4] feat: change animation creation to loaded, thus it works
 in popup.

---
 .../Controls/SelectionList/SelectionList.cs   | 69 +++++++++----------
 1 file changed, 31 insertions(+), 38 deletions(-)

diff --git a/src/Ursa/Controls/SelectionList/SelectionList.cs b/src/Ursa/Controls/SelectionList/SelectionList.cs
index cbce4ea..045421c 100644
--- a/src/Ursa/Controls/SelectionList/SelectionList.cs
+++ b/src/Ursa/Controls/SelectionList/SelectionList.cs
@@ -14,33 +14,34 @@ using Irihi.Avalonia.Shared.Helpers;
 namespace Ursa.Controls;
 
 [TemplatePart(PART_Indicator, typeof(ContentPresenter))]
-public class SelectionList: SelectingItemsControl
+public class SelectionList : SelectingItemsControl
 {
     public const string PART_Indicator = "PART_Indicator";
     private static readonly FuncTemplate<Panel?> DefaultPanel = new(() => new StackPanel());
-    
+
+    public static readonly StyledProperty<Control?> IndicatorProperty =
+        AvaloniaProperty.Register<SelectionList, Control?>(
+            nameof(Indicator));
+
     private ImplicitAnimationCollection? _implicitAnimations;
     private ContentPresenter? _indicator;
 
-    public static readonly StyledProperty<Control?> IndicatorProperty = AvaloniaProperty.Register<SelectionList, Control?>(
-        nameof(Indicator));
+    static SelectionList()
+    {
+        SelectionModeProperty.OverrideMetadata<SelectionList>(
+            new StyledPropertyMetadata<SelectionMode>(
+                SelectionMode.Single,
+                coerce: (_, _) => SelectionMode.Single)
+        );
+        SelectedItemProperty.Changed.AddClassHandler<SelectionList, object?>((list, args) =>
+            list.OnSelectedItemChanged(args));
+    }
 
     public Control? Indicator
     {
         get => GetValue(IndicatorProperty);
         set => SetValue(IndicatorProperty, value);
     }
-    
-    static SelectionList()
-    {
-        SelectionModeProperty.OverrideMetadata<SelectionList>(
-            new StyledPropertyMetadata<SelectionMode>(
-            defaultValue: SelectionMode.Single, 
-            coerce: (_, _) => SelectionMode.Single)
-            );
-        SelectedItemProperty.Changed.AddClassHandler<SelectionList, object?>((list, args) =>
-            list.OnSelectedItemChanged(args));
-    }
 
     private void OnSelectedItemChanged(AvaloniaPropertyChangedEventArgs<object?> args)
     {
@@ -50,12 +51,14 @@ public class SelectionList: SelectingItemsControl
             OpacityProperty.SetValue(0d, _indicator);
             return;
         }
+
         var container = ContainerFromItem(newValue);
         if (container is null)
         {
             OpacityProperty.SetValue(0d, _indicator);
             return;
         }
+
         OpacityProperty.SetValue(1d, _indicator);
         InvalidateMeasure();
         // InvalidateArrange();
@@ -64,15 +67,12 @@ public class SelectionList: SelectingItemsControl
     protected override Size ArrangeOverride(Size finalSize)
     {
         var size = base.ArrangeOverride(finalSize);
-        if(_indicator is not null && SelectedItem is not null)
+        if (_indicator is not null && SelectedItem is not null)
         {
             var container = ContainerFromItem(SelectedItem);
             if (container is null) return size;
             var bounds = container.Bounds;
-            if (ItemsPanelRoot?.Bounds.Position is { } p)
-            {
-                bounds = bounds.Translate(p);
-            }
+            if (ItemsPanelRoot?.Bounds.Position is { } p) bounds = bounds.Translate(p);
             _indicator.Arrange(bounds);
         }
         else
@@ -80,6 +80,7 @@ public class SelectionList: SelectingItemsControl
             // This is a hack. The indicator is not visible, so we arrange it to a 1x1 rectangle
             _indicator?.Arrange(new Rect(new Point(), new Size(1, 1)));
         }
+
         return size;
     }
 
@@ -96,23 +97,20 @@ public class SelectionList: SelectingItemsControl
     protected override void OnApplyTemplate(TemplateAppliedEventArgs e)
     {
         base.OnApplyTemplate(e);
-        _indicator= e.NameScope.Find<ContentPresenter>(PART_Indicator);
-        EnsureIndicatorAnimation();
+        _indicator = e.NameScope.Find<ContentPresenter>(PART_Indicator);
     }
 
     protected override void OnLoaded(RoutedEventArgs e)
     {
         base.OnLoaded(e);
-        if(_indicator is not null && SelectedItem is not null)
+        EnsureIndicatorAnimation();
+        if (_indicator is not null && SelectedItem is not null)
         {
             var container = ContainerFromItem(SelectedItem);
             if (container is null) return;
             _indicator.Opacity = 1;
             var bounds = container.Bounds;
-            if (ItemsPanelRoot?.Bounds.Position is { } p)
-            {
-                bounds = bounds.Translate(p);
-            }
+            if (ItemsPanelRoot?.Bounds.Position is { } p) bounds = bounds.Translate(p);
             _indicator.Arrange(bounds);
         }
     }
@@ -123,10 +121,7 @@ public class SelectionList: SelectingItemsControl
         {
             _indicator.Opacity = 0;
             SetUpAnimation();
-            if (ElementComposition.GetElementVisual(_indicator) is { } v)
-            {
-                v.ImplicitAnimations = _implicitAnimations;
-            }
+            if (ElementComposition.GetElementVisual(_indicator) is { } v) v.ImplicitAnimations = _implicitAnimations;
         }
     }
 
@@ -136,7 +131,7 @@ public class SelectionList: SelectingItemsControl
         Selection.Clear();
         Selection.Select(index);
     }
-    
+
     private void SetUpAnimation()
     {
         if (_implicitAnimations != null) return;
@@ -161,15 +156,13 @@ public class SelectionList: SelectingItemsControl
         _implicitAnimations[nameof(CompositionVisual.Size)] = sizeAnimation;
         _implicitAnimations[nameof(CompositionVisual.Opacity)] = opacityAnimation;
     }
-    
+
     protected override void OnKeyDown(KeyEventArgs e)
     {
         var hotkeys = Application.Current!.PlatformSettings?.HotkeyConfiguration;
-        
-        if (e.Key.ToNavigationDirection() is { } direction &&  direction.IsDirectional())
-        {
+
+        if (e.Key.ToNavigationDirection() is { } direction && direction.IsDirectional())
             e.Handled |= MoveSelection(direction, WrapSelection);
-        }
         base.OnKeyDown(e);
     }
-}
+}
\ No newline at end of file

From 1e20c81b7d81d3be27eafb79e2cfbf2118f3bde6 Mon Sep 17 00:00:00 2001
From: Dong Bin <popmessiah@hotmail.com>
Date: Tue, 25 Feb 2025 01:42:26 +0800
Subject: [PATCH 4/4] misc: add flyout demo.

---
 demo/Ursa.Demo/Pages/SelectionListDemo.axaml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/demo/Ursa.Demo/Pages/SelectionListDemo.axaml b/demo/Ursa.Demo/Pages/SelectionListDemo.axaml
index ec4a9c3..242791b 100644
--- a/demo/Ursa.Demo/Pages/SelectionListDemo.axaml
+++ b/demo/Ursa.Demo/Pages/SelectionListDemo.axaml
@@ -54,6 +54,14 @@
                 </DataTemplate>
             </u:SelectionList.ItemTemplate>
         </u:SelectionList>
-        <Button Command="{Binding Clear}">Clear</Button>
+        <SplitButton Content="Button With Selection">
+            <SplitButton.Flyout>
+                <Flyout>
+                    <u:SelectionList ItemsSource="{Binding Items}" SelectedItem="{Binding SelectedItem}" />
+                </Flyout>
+            </SplitButton.Flyout>
+        </SplitButton>
+        <Button Margin="0 12" Command="{Binding Clear}">Clear</Button>
+        
     </StackPanel>
 </UserControl>