From d432fcc68c42025e6cffa57910bd97e2901efa2b Mon Sep 17 00:00:00 2001
From: punker76 <punkerat76@gmail.com>
Date: Mon, 26 Feb 2024 23:11:28 +0100
Subject: [PATCH] fix: #4466 collapse colon between hours and minute of
 timepicker if hours collapsed

Add also a new flag enum to show only minute and seconds.
---
 .../ExampleViews/DateExamples.xaml            | 27 ++++-------------
 .../Controls/TimePicker/TimePartVisibility.cs |  1 +
 src/MahApps.Metro/Themes/DateTimePicker.xaml  | 29 +++++++++++++++----
 3 files changed, 29 insertions(+), 28 deletions(-)

diff --git a/src/MahApps.Metro.Samples/MahApps.Metro.Demo/ExampleViews/DateExamples.xaml b/src/MahApps.Metro.Samples/MahApps.Metro.Demo/ExampleViews/DateExamples.xaml
index a7206ce82f..09ba9013f7 100644
--- a/src/MahApps.Metro.Samples/MahApps.Metro.Demo/ExampleViews/DateExamples.xaml
+++ b/src/MahApps.Metro.Samples/MahApps.Metro.Demo/ExampleViews/DateExamples.xaml
@@ -9,6 +9,7 @@
              xmlns:mah="http://metro.mahapps.com/winfx/xaml/controls"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
              xmlns:system="clr-namespace:System;assembly=mscorlib"
+             xmlns:markup="clr-namespace:MetroDemo.Markup"
              d:DataContext="{d:DesignInstance local:MainWindowViewModel}"
              d:DesignHeight="600"
              d:DesignWidth="800"
@@ -159,17 +160,8 @@
                                       Grid.Row="1"
                                       Grid.Column="1"
                                       Margin="{StaticResource ControlMargin}"
-                                      SelectedItem="{x:Static mah:TimePartVisibility.All}">
-                                <ComboBox.ItemsSource>
-                                    <x:Array Type="mah:TimePartVisibility">
-                                        <mah:TimePartVisibility>All</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>HourMinute</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>Hour</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>Minute</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>Second</mah:TimePartVisibility>
-                                    </x:Array>
-                                </ComboBox.ItemsSource>
-                            </ComboBox>
+                                      ItemsSource="{Binding Source={markup:EnumBindingSource {x:Type mah:TimePartVisibility}}, Mode=OneTime}"
+                                      SelectedItem="{x:Static mah:TimePartVisibility.All}" />
 
                             <Label Grid.Row="2"
                                    Grid.Column="0"
@@ -179,17 +171,8 @@
                                       Grid.Row="2"
                                       Grid.Column="1"
                                       Margin="{StaticResource ControlMargin}"
-                                      SelectedItem="{x:Static mah:TimePartVisibility.All}">
-                                <ComboBox.ItemsSource>
-                                    <x:Array Type="mah:TimePartVisibility">
-                                        <mah:TimePartVisibility>All</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>HourMinute</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>Hour</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>Minute</mah:TimePartVisibility>
-                                        <mah:TimePartVisibility>Second</mah:TimePartVisibility>
-                                    </x:Array>
-                                </ComboBox.ItemsSource>
-                            </ComboBox>
+                                      ItemsSource="{Binding Source={markup:EnumBindingSource {x:Type mah:TimePartVisibility}}, Mode=OneTime}"
+                                      SelectedItem="{x:Static mah:TimePartVisibility.All}" />
 
                             <Label Grid.Row="3"
                                    Grid.Column="0"
diff --git a/src/MahApps.Metro/Controls/TimePicker/TimePartVisibility.cs b/src/MahApps.Metro/Controls/TimePicker/TimePartVisibility.cs
index a2043d9711..f36d075537 100644
--- a/src/MahApps.Metro/Controls/TimePicker/TimePartVisibility.cs
+++ b/src/MahApps.Metro/Controls/TimePicker/TimePartVisibility.cs
@@ -16,6 +16,7 @@ public enum TimePartVisibility
         Minute = 1 << 2,
         Second = 1 << 3,
         HourMinute = Hour | Minute,
+        MinuteSecond = Minute | Second,
         All = HourMinute | Second
     }
 }
\ No newline at end of file
diff --git a/src/MahApps.Metro/Themes/DateTimePicker.xaml b/src/MahApps.Metro/Themes/DateTimePicker.xaml
index 895095cd7e..4c6a1d26c3 100644
--- a/src/MahApps.Metro/Themes/DateTimePicker.xaml
+++ b/src/MahApps.Metro/Themes/DateTimePicker.xaml
@@ -354,18 +354,20 @@
                                                           FontSize="{TemplateBinding FontSize}"
                                                           ItemStringFormat="{TemplateBinding HoursItemStringFormat}"
                                                           ItemsSource="{TemplateBinding SourceHours}" />
-                                                <Label Grid.Column="1"
-                                                       Content=":"
-                                                       Visibility="{Binding Path=Visibility, ElementName=PART_MinutePicker}" />
+                                                <Label x:Name="PART_HourMinuteColon"
+                                                       Grid.Column="1"
+                                                       Visibility="Collapsed"
+                                                       Content=":" />
                                                 <ComboBox x:Name="PART_MinutePicker"
                                                           Grid.Column="2"
                                                           FontFamily="{TemplateBinding FontFamily}"
                                                           FontSize="{TemplateBinding FontSize}"
                                                           ItemStringFormat="{TemplateBinding MinutesItemStringFormat}"
                                                           ItemsSource="{TemplateBinding SourceMinutes}" />
-                                                <Label Grid.Column="3"
-                                                       Content=":"
-                                                       Visibility="{Binding Path=Visibility, ElementName=PART_SecondPicker}" />
+                                                <Label x:Name="PART_MinuteSecondColon"
+                                                       Grid.Column="3"
+                                                       Visibility="Collapsed"
+                                                       Content=":" />
                                                 <ComboBox x:Name="PART_SecondPicker"
                                                           Grid.Column="4"
                                                           FontFamily="{TemplateBinding FontFamily}"
@@ -395,6 +397,21 @@
 
                     </Grid>
                     <ControlTemplate.Triggers>
+                        <MultiTrigger>
+                            <MultiTrigger.Conditions>
+                                <Condition SourceName="PART_HourPicker" Property="Visibility" Value="Visible" />
+                                <Condition SourceName="PART_MinutePicker" Property="Visibility" Value="Visible" />
+                            </MultiTrigger.Conditions>
+                            <Setter TargetName="PART_HourMinuteColon" Property="Visibility" Value="Visible" />
+                        </MultiTrigger>
+                        <MultiTrigger>
+                            <MultiTrigger.Conditions>
+                                <Condition SourceName="PART_MinutePicker" Property="Visibility" Value="Visible" />
+                                <Condition SourceName="PART_SecondPicker" Property="Visibility" Value="Visible" />
+                            </MultiTrigger.Conditions>
+                            <Setter TargetName="PART_MinuteSecondColon" Property="Visibility" Value="Visible" />
+                        </MultiTrigger>
+
                         <Trigger Property="mah:TextBoxHelper.ClearTextButton" Value="True">
                             <Setter TargetName="PART_ClearText" Property="Command" Value="{x:Static mah:MahAppsCommands.ClearControlCommand}" />
                         </Trigger>