From 81b3635a2d32874987c82e6cb6ac104480b99ff9 Mon Sep 17 00:00:00 2001
From: Zhang Dian <54255897+zdpcdt@users.noreply.github.com>
Date: Sun, 19 Jan 2025 18:09:29 +0800
Subject: [PATCH 1/3] fix: fix various issues.

* fix CornerRadius of CloseButton.
* add IsVisible to Content of DualBadge.
* remove Geometry & sys:.
---
 .../Controls/DialogShared.axaml               |  2 +-
 src/Ursa.Themes.Semi/Controls/DualBadge.axaml |  1 +
 .../Themes/Shared/NavigationMenu.axaml        | 30 +++++++++----------
 3 files changed, 16 insertions(+), 17 deletions(-)

diff --git a/src/Ursa.Themes.Semi/Controls/DialogShared.axaml b/src/Ursa.Themes.Semi/Controls/DialogShared.axaml
index 4ddc3a6..bcaf861 100644
--- a/src/Ursa.Themes.Semi/Controls/DialogShared.axaml
+++ b/src/Ursa.Themes.Semi/Controls/DialogShared.axaml
@@ -9,7 +9,7 @@
     </Design.PreviewWith>
 
     <ControlTheme x:Key="OverlayCloseButton" TargetType="Button">
-        <Setter Property="CornerRadius" Value="6" />
+        <Setter Property="CornerRadius" Value="3" />
         <Setter Property="Height" Value="24" />
         <Setter Property="Width" Value="24" />
         <Setter Property="Padding" Value="4" />
diff --git a/src/Ursa.Themes.Semi/Controls/DualBadge.axaml b/src/Ursa.Themes.Semi/Controls/DualBadge.axaml
index 79859b1..2e2a3fb 100644
--- a/src/Ursa.Themes.Semi/Controls/DualBadge.axaml
+++ b/src/Ursa.Themes.Semi/Controls/DualBadge.axaml
@@ -49,6 +49,7 @@
                             Name="{x:Static u:DualBadge.PART_ContentPresenter}"
                             Grid.Column="1"
                             TextElement.FontSize="{TemplateBinding FontSize}"
+                            IsVisible="{TemplateBinding Content,Converter={x:Static ObjectConverters.IsNotNull}}"
                             Foreground="{TemplateBinding Foreground}"
                             Background="{TemplateBinding Background}"
                             Padding="{TemplateBinding Padding}"
diff --git a/src/Ursa.Themes.Semi/Themes/Shared/NavigationMenu.axaml b/src/Ursa.Themes.Semi/Themes/Shared/NavigationMenu.axaml
index 7b8ec35..e6de440 100644
--- a/src/Ursa.Themes.Semi/Themes/Shared/NavigationMenu.axaml
+++ b/src/Ursa.Themes.Semi/Themes/Shared/NavigationMenu.axaml
@@ -1,25 +1,23 @@
 <ResourceDictionary
     xmlns="https://github.com/avaloniaui"
-    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
-    xmlns:sys="clr-namespace:System;assembly=netstandard">
-    <Geometry x:Key="NavigationMenuExpandIconGlyph">M5 2H19C20.6569 2 22 3.34315 22 5V19C22 20.6569 20.6569 22 19 22H5C3.34315 22 2 20.6569 2 19V5C2 3.34315 3.34315 2 5 2ZM6 4C5.44772 4 5 4.44772 5 5V19C5 19.5523 5.44772 20 6 20H9C9.55229 20 10 19.5523 10 19V5C10 4.44772 9.55229 4 9 4H6Z</Geometry>
-    <Geometry x:Key="NavigationMenuItemExpandIconGlyph">M4.08045 7.59809C4.66624 7.01231 5.61599 7.01231 6.20177 7.59809L11.8586 13.2549L17.5155 7.59809C18.1013 7.01231 19.051 7.01231 19.6368 7.59809C20.2226 8.18388 20.2226 9.13363 19.6368 9.71941L12.9193 16.4369C12.3335 17.0227 11.3838 17.0227 10.798 16.4369L4.08045 9.71941C3.49467 9.13363 3.49467 8.18388 4.08045 7.59809Z</Geometry>
+    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml">
+    <StaticResource x:Key="NavigationMenuItemExpandIconGlyph" ResourceKey="SemiIconChevronDown" />
 
-    <sys:Double x:Key="NavigationMenuExpandWidth">260</sys:Double>
-    <sys:Double x:Key="NavigationMenuClosedWidth">80</sys:Double>
-    <sys:Double x:Key="NavigationMenuExpandIconWidth">16</sys:Double>
-    <sys:Double x:Key="NavigationMenuExpandIconHeight">16</sys:Double>
+    <x:Double x:Key="NavigationMenuExpandWidth">260</x:Double>
+    <x:Double x:Key="NavigationMenuClosedWidth">80</x:Double>
+    <x:Double x:Key="NavigationMenuExpandIconWidth">16</x:Double>
+    <x:Double x:Key="NavigationMenuExpandIconHeight">16</x:Double>
 
-    <sys:Double x:Key="NavigationMenuItemIconMinWidth">24</sys:Double>
-    <sys:Double x:Key="NavigationMenuItemExpandIconWidth">8</sys:Double>
-    <sys:Double x:Key="NavigationMenuItemExpandIconHeight">8</sys:Double>
+    <x:Double x:Key="NavigationMenuItemIconMinWidth">24</x:Double>
+    <x:Double x:Key="NavigationMenuItemExpandIconWidth">8</x:Double>
+    <x:Double x:Key="NavigationMenuItemExpandIconHeight">8</x:Double>
     <Thickness x:Key="NavigationMenuItemIconMargin">0</Thickness>
-    <sys:Double x:Key="NavigationMenuItemCollapsedHeight">0</sys:Double>
-    <sys:Double x:Key="NavigationMenuItemCollapsedOpacity">0</sys:Double>
+    <x:Double x:Key="NavigationMenuItemCollapsedHeight">0</x:Double>
+    <x:Double x:Key="NavigationMenuItemCollapsedOpacity">0</x:Double>
 
-    <sys:Double x:Key="NavigationMenuSeparatorHeaderFontSize">12</sys:Double>
-    <sys:Double x:Key="NavigationMenuSeparatorBorderHeight">1</sys:Double>
-    <sys:Double x:Key="NavigationMenuSeparatorClosedWidth">32</sys:Double>
+    <x:Double x:Key="NavigationMenuSeparatorHeaderFontSize">12</x:Double>
+    <x:Double x:Key="NavigationMenuSeparatorBorderHeight">1</x:Double>
+    <x:Double x:Key="NavigationMenuSeparatorClosedWidth">32</x:Double>
 
 
     <Thickness x:Key="NavigationMenuHeaderMargin">12</Thickness>

From b5489062afcaaca358ee12f8a6835f0484a47bb9 Mon Sep 17 00:00:00 2001
From: Zhang Dian <54255897+zdpcdt@users.noreply.github.com>
Date: Sun, 19 Jan 2025 18:12:58 +0800
Subject: [PATCH 2/3] misc: using InnerPathIcon.

---
 src/Ursa.Themes.Semi/Controls/Calendar.axaml  | 16 ++++++-------
 src/Ursa.Themes.Semi/Controls/NavMenu.axaml   |  3 +--
 .../Controls/Pagination.axaml                 | 24 +++++++------------
 .../Controls/ScrollToButton.axaml             |  8 +++----
 src/Ursa.Themes.Semi/Controls/TagInput.axaml  |  4 ++--
 5 files changed, 22 insertions(+), 33 deletions(-)

diff --git a/src/Ursa.Themes.Semi/Controls/Calendar.axaml b/src/Ursa.Themes.Semi/Controls/Calendar.axaml
index 27a1a50..72f21f6 100644
--- a/src/Ursa.Themes.Semi/Controls/Calendar.axaml
+++ b/src/Ursa.Themes.Semi/Controls/Calendar.axaml
@@ -206,8 +206,8 @@
                             Foreground="{TemplateBinding Foreground}"
                             Theme="{DynamicResource BorderlessButton}">
                             <PathIcon
-                                Width="{DynamicResource CalenderViewPathIconWidth}"
-                                Height="{DynamicResource CalenderViewPathIconHeight}"
+                                Theme="{StaticResource InnerPathIcon}"
+                                Classes="Large"
                                 Data="{DynamicResource CalendarViewFastForwardGlyph}"
                                 Foreground="{DynamicResource CalendarItemIconForeground}" />
                         </Button>
@@ -219,8 +219,8 @@
                             Foreground="{TemplateBinding Foreground}"
                             Theme="{DynamicResource BorderlessButton}">
                             <PathIcon
-                                Width="{DynamicResource CalenderViewPathIconWidth}"
-                                Height="{DynamicResource CalenderViewPathIconHeight}"
+                                Theme="{StaticResource InnerPathIcon}"
+                                Classes="Large"
                                 Data="{DynamicResource CalendarItemPreviousIconGlyph}"
                                 Foreground="{DynamicResource CalendarItemIconForeground}" />
                         </Button>
@@ -253,8 +253,8 @@
                             Foreground="{TemplateBinding Foreground}"
                             Theme="{DynamicResource BorderlessButton}">
                             <PathIcon
-                                Width="{DynamicResource CalenderViewPathIconWidth}"
-                                Height="{DynamicResource CalenderViewPathIconHeight}"
+                                Theme="{StaticResource InnerPathIcon}"
+                                Classes="Large"
                                 Data="{DynamicResource CalendarItemNextIconGlyph}"
                                 Foreground="{DynamicResource CalendarItemIconForeground}" />
                         </Button>
@@ -265,8 +265,8 @@
                             Foreground="{TemplateBinding Foreground}"
                             Theme="{DynamicResource BorderlessButton}">
                             <PathIcon
-                                Width="{DynamicResource CalenderViewPathIconWidth}"
-                                Height="{DynamicResource CalenderViewPathIconHeight}"
+                                Theme="{StaticResource InnerPathIcon}"
+                                Classes="Large"
                                 Data="{DynamicResource CalendarViewFastBackwardGlyph}"
                                 Foreground="{DynamicResource CalendarItemIconForeground}" />
                         </Button>
diff --git a/src/Ursa.Themes.Semi/Controls/NavMenu.axaml b/src/Ursa.Themes.Semi/Controls/NavMenu.axaml
index d0eea3e..9f8a54a 100644
--- a/src/Ursa.Themes.Semi/Controls/NavMenu.axaml
+++ b/src/Ursa.Themes.Semi/Controls/NavMenu.axaml
@@ -91,8 +91,7 @@
                     <PathIcon
                         Name="PART_ExpanderIcon"
                         Grid.Column="2"
-                        Width="8"
-                        Height="8"
+                        Theme="{StaticResource InnerPathIcon}"
                         Margin="12,0"
                         Data="{DynamicResource NavigationMenuItemExpandIconGlyph}"
                         Foreground="{DynamicResource NavigationMenuItemExpandIconForeground}">
diff --git a/src/Ursa.Themes.Semi/Controls/Pagination.axaml b/src/Ursa.Themes.Semi/Controls/Pagination.axaml
index 558bede..401e286 100644
--- a/src/Ursa.Themes.Semi/Controls/Pagination.axaml
+++ b/src/Ursa.Themes.Semi/Controls/Pagination.axaml
@@ -14,16 +14,14 @@
                 <StackPanel Orientation="Horizontal">
                     <u:PaginationButton Name="{x:Static u:Pagination.PART_PreviousButton}" u:DisabledAdorner.IsEnabled="True">
                         <PathIcon
-                            Width="12"
-                            Height="12"
+                            Theme="{StaticResource InnerPathIcon}"
                             Data="{DynamicResource PaginationBackwardGlyph}"
                             Foreground="{DynamicResource PaginationButtonIconForeground}" />
                     </u:PaginationButton>
                     <StackPanel Name="{x:Static u:Pagination.PART_ButtonPanel}" Orientation="Horizontal" />
                     <u:PaginationButton Name="{x:Static u:Pagination.PART_NextButton}" u:DisabledAdorner.IsEnabled="True">
                         <PathIcon
-                            Width="12"
-                            Height="12"
+                            Theme="{StaticResource InnerPathIcon}"
                             Data="{DynamicResource PaginationForwardGlyph}"
                             Foreground="{DynamicResource PaginationButtonIconForeground}" />
                     </u:PaginationButton>
@@ -58,8 +56,7 @@
                 <StackPanel Orientation="Horizontal">
                     <u:PaginationButton Name="{x:Static u:Pagination.PART_PreviousButton}" u:DisabledAdorner.IsEnabled="True">
                         <PathIcon
-                            Width="12"
-                            Height="12"
+                            Theme="{StaticResource InnerPathIcon}"
                             Data="{DynamicResource PaginationBackwardGlyph}"
                             Foreground="{DynamicResource PaginationButtonIconForeground}" />
                     </u:PaginationButton>
@@ -92,8 +89,7 @@
                     </Grid>
                     <u:PaginationButton Name="{x:Static u:Pagination.PART_NextButton}" u:DisabledAdorner.IsEnabled="True">
                         <PathIcon
-                            Width="12"
-                            Height="12"
+                            Theme="{StaticResource InnerPathIcon}"
                             Data="{DynamicResource PaginationForwardGlyph}"
                             Foreground="{DynamicResource PaginationButtonIconForeground}" />
                     </u:PaginationButton>
@@ -143,8 +139,7 @@
             <Setter Property="Content">
                 <Template>
                     <PathIcon
-                        Width="12"
-                        Height="12"
+                        Theme="{StaticResource InnerPathIcon}"
                         Data="{DynamicResource PaginationMoreGlyph}"
                         Foreground="{DynamicResource PaginationButtonIconForeground}" />
                 </Template>
@@ -153,8 +148,7 @@
                 <Setter Property="Content">
                     <Template>
                         <PathIcon
-                            Width="12"
-                            Height="12"
+                            Theme="{StaticResource InnerPathIcon}"
                             Data="{DynamicResource PaginationFastBackwardGlyph}"
                             Foreground="{DynamicResource PaginationButtonIconForeground}" />
                     </Template>
@@ -165,8 +159,7 @@
             <Setter Property="Content">
                 <Template>
                     <PathIcon
-                        Width="12"
-                        Height="12"
+                        Theme="{StaticResource InnerPathIcon}"
                         Data="{DynamicResource PaginationMoreGlyph}"
                         Foreground="{DynamicResource PaginationButtonIconForeground}" />
                 </Template>
@@ -175,8 +168,7 @@
                 <Setter Property="Content">
                     <Template>
                         <PathIcon
-                            Width="12"
-                            Height="12"
+                            Theme="{StaticResource InnerPathIcon}"
                             Data="{DynamicResource PaginationFastForwardGlyph}"
                             Foreground="{DynamicResource PaginationButtonIconForeground}" />
                     </Template>
diff --git a/src/Ursa.Themes.Semi/Controls/ScrollToButton.axaml b/src/Ursa.Themes.Semi/Controls/ScrollToButton.axaml
index e0859d3..67c8c64 100644
--- a/src/Ursa.Themes.Semi/Controls/ScrollToButton.axaml
+++ b/src/Ursa.Themes.Semi/Controls/ScrollToButton.axaml
@@ -17,8 +17,7 @@
                     <PathIcon
                         Name="PART_Icon"
                         Margin="8"
-                        Width="12"
-                        Height="12"
+                        Theme="{StaticResource InnerPathIcon}"
                         Data="{DynamicResource ScrollToButtonIconGlyph}"
                         Foreground="{DynamicResource ButtonDefaultPrimaryForeground}" />
                 </Border>
@@ -61,8 +60,7 @@
                     <PathIcon
                         Name="PART_Icon"
                         Margin="8"
-                        Width="12"
-                        Height="12"
+                        Theme="{StaticResource InnerPathIcon}"
                         Data="{DynamicResource ScrollToButtonIconGlyph}"
                         Foreground="{DynamicResource ButtonSolidForeground}" />
                 </Border>
@@ -89,4 +87,4 @@
             <Setter Property="RenderTransform" Value="rotate(270deg)" />
         </Style>
     </ControlTheme>
-</ResourceDictionary>
+</ResourceDictionary>
\ No newline at end of file
diff --git a/src/Ursa.Themes.Semi/Controls/TagInput.axaml b/src/Ursa.Themes.Semi/Controls/TagInput.axaml
index e3a8732..4247894 100644
--- a/src/Ursa.Themes.Semi/Controls/TagInput.axaml
+++ b/src/Ursa.Themes.Semi/Controls/TagInput.axaml
@@ -124,8 +124,8 @@
                     <DockPanel LastChildFill="True">
                         <PathIcon
                             Name="{x:Static u:ClosableTag.PART_CloseButton}"
-                            Width="8"
-                            Height="8"
+                            Theme="{StaticResource InnerPathIcon}"
+                            Classes="Small"
                             Margin="4,0"
                             Background="Transparent"
                             Data="{DynamicResource ClosableTagCloseIconGlyph}"

From 969661e121c20bc36c5e6f7e5ce66627a8044fdf Mon Sep 17 00:00:00 2001
From: Zhang Dian <54255897+zdpcdt@users.noreply.github.com>
Date: Mon, 20 Jan 2025 00:30:17 +0800
Subject: [PATCH 3/3] misc: add Width & Height to Button in ThemeSelector &
 TitleBar.

---
 demo/Ursa.Demo/Views/TitleBarRightContent.axaml   | 2 ++
 src/Ursa.Themes.Semi/Controls/ThemeSelector.axaml | 9 +++++----
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/demo/Ursa.Demo/Views/TitleBarRightContent.axaml b/demo/Ursa.Demo/Views/TitleBarRightContent.axaml
index 1ba3aaa..4fa97bf 100644
--- a/demo/Ursa.Demo/Views/TitleBarRightContent.axaml
+++ b/demo/Ursa.Demo/Views/TitleBarRightContent.axaml
@@ -8,6 +8,8 @@
     <StackPanel Orientation="Horizontal">
         <Button
             Padding="8"
+            Width="32"
+            Height="32"
             Theme="{DynamicResource BorderlessButton}"
             Click="OpenRepository">
             <PathIcon
diff --git a/src/Ursa.Themes.Semi/Controls/ThemeSelector.axaml b/src/Ursa.Themes.Semi/Controls/ThemeSelector.axaml
index 204768a..e012588 100644
--- a/src/Ursa.Themes.Semi/Controls/ThemeSelector.axaml
+++ b/src/Ursa.Themes.Semi/Controls/ThemeSelector.axaml
@@ -2,14 +2,15 @@
                     xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
                     xmlns:u="https://irihi.tech/ursa">
     <ControlTheme TargetType="u:ThemeToggleButton" x:Key="{x:Type u:ThemeToggleButton}">
+        <Setter Property="Padding" Value="8" />
+        <Setter Property="Width" Value="32" />
+        <Setter Property="Height" Value="32" />
         <Setter Property="Template">
             <ControlTemplate TargetType="u:ThemeToggleButton">
                 <Button
-                    Padding="8"
-                    FontWeight="Regular"
                     Name="{x:Static u:ThemeToggleButton.PART_ThemeButton}"
-                    HorizontalAlignment="Center"
-                    Theme="{DynamicResource BorderlessButton}">
+                    Theme="{DynamicResource BorderlessButton}"
+                    Padding="{TemplateBinding Padding}">
                     <PathIcon
                         Name="PART_Icon"
                         Theme="{StaticResource InnerPathIcon}"