From dfa35dc5a25b5c3262b544f9d4117743e4934059 Mon Sep 17 00:00:00 2001
From: Marcel Wagner <marcel.alex.wagner@outlook.com>
Date: Mon, 31 Jan 2022 01:14:09 +0100
Subject: [PATCH] Add separator between code samples, update code copy button
 to be on title line (#854)

---
 XamlControlsGallery/Controls/ControlExample.xaml    | 13 ++++++++++++-
 XamlControlsGallery/Controls/ControlExample.xaml.cs | 11 +++++++++--
 .../Controls/SampleCodePresenter.xaml               |  4 ++--
 .../Controls/SampleCodePresenter.xaml.cs            |  2 ++
 4 files changed, 25 insertions(+), 5 deletions(-)

diff --git a/XamlControlsGallery/Controls/ControlExample.xaml b/XamlControlsGallery/Controls/ControlExample.xaml
index 58088c3a5..b56f49518 100644
--- a/XamlControlsGallery/Controls/ControlExample.xaml
+++ b/XamlControlsGallery/Controls/ControlExample.xaml
@@ -60,6 +60,15 @@
                     </VisualState.Setters>
                 </VisualState>
             </VisualStateGroup>
+
+            <VisualStateGroup x:Name="SampleCodeSeparatorStates">
+                <VisualState x:Name="SeparatorCollapsed"/>
+                <VisualState x:Name="SeparatorVisible">
+                    <VisualState.Setters>
+                        <Setter Target="SampleCodeSeparator.Visibility" Value="Visible"/>
+                    </VisualState.Setters>
+                </VisualState>
+            </VisualStateGroup>
         </VisualStateManager.VisualStateGroups>
 
         <Grid.RowDefinitions>
@@ -154,12 +163,14 @@
                     <TextBlock Text="Source code"/>
                 </muxc:Expander.Header>
                 
-                <StackPanel x:DefaultBindMode="OneWay" Spacing="4">
+                <StackPanel x:DefaultBindMode="OneWay">
                     <controls:SampleCodePresenter
                         x:Name="XamlPresenter"
                         Code="{x:Bind Xaml}"
                         CodeSourceFile="{x:Bind XamlSource}"
                         Substitutions="{x:Bind Substitutions}"/>
+                    <Border x:Name="SampleCodeSeparator" Visibility="Collapsed" Margin="0,20"
+                        BorderThickness="1" BorderBrush="{ThemeResource SystemControlBackgroundBaseMediumBrush}"/>
                     <controls:SampleCodePresenter
                         x:Name="CSharpPresenter"
                         Code="{x:Bind CSharp}"
diff --git a/XamlControlsGallery/Controls/ControlExample.xaml.cs b/XamlControlsGallery/Controls/ControlExample.xaml.cs
index 60014d6ff..a9305db05 100644
--- a/XamlControlsGallery/Controls/ControlExample.xaml.cs
+++ b/XamlControlsGallery/Controls/ControlExample.xaml.cs
@@ -188,14 +188,21 @@ public int MinimumUniversalAPIContract
             set { SetValue(MinimumUniversalAPIContractProperty, value); }
         }
 
-        private ILanguage CSharpLanguage = Languages.CSharp;
-
         public ControlExample()
         {
             this.InitializeComponent();
             Substitutions = new List<ControlExampleSubstitution>();
 
             ControlPresenter.RegisterPropertyChangedCallback(ContentPresenter.PaddingProperty, ControlPaddingChangedCallback);
+            this.Loaded += ControlExample_Loaded;
+        }
+
+        private void ControlExample_Loaded(object sender, RoutedEventArgs e)
+        {
+            if(!XamlPresenter.IsEmpty && !CSharpPresenter.IsEmpty)
+            {
+                VisualStateManager.GoToState(this, "SeparatorVisible", false);
+            }
         }
 
         private void rootGrid_Loaded(object sender, RoutedEventArgs e)
diff --git a/XamlControlsGallery/Controls/SampleCodePresenter.xaml b/XamlControlsGallery/Controls/SampleCodePresenter.xaml
index 1169ef332..e80ec973d 100644
--- a/XamlControlsGallery/Controls/SampleCodePresenter.xaml
+++ b/XamlControlsGallery/Controls/SampleCodePresenter.xaml
@@ -41,14 +41,14 @@
         </VisualStateManager.VisualStateGroups>
         <TextBlock x:Name="SampleHeader"/>
         <ScrollViewer
-                x:Name="CodeScrollViewer" Padding="0,4,32,10"
+                x:Name="CodeScrollViewer" Padding="0,4,0,10"
                 HorizontalScrollBarVisibility="Auto" Grid.Row="1"
                 HorizontalScrollMode="Auto"
                 VerticalScrollBarVisibility="Disabled"
                 VerticalScrollMode="Disabled">
             <ContentPresenter x:Name="CodePresenter" Loaded="CodePresenter_Loaded"/>
         </ScrollViewer>
-        <Button x:Name="CopyCodeButton" Grid.Row="1"
+        <Button x:Name="CopyCodeButton" Grid.Row="0"
                 AutomationProperties.Name="Copy sample code"
                 ToolTipService.ToolTip="Copy sample code" VerticalAlignment="Top"
                 HorizontalAlignment="Right" Click="CopyCodeButton_Click" Padding="4">
diff --git a/XamlControlsGallery/Controls/SampleCodePresenter.xaml.cs b/XamlControlsGallery/Controls/SampleCodePresenter.xaml.cs
index 3628553f8..5a865baae 100644
--- a/XamlControlsGallery/Controls/SampleCodePresenter.xaml.cs
+++ b/XamlControlsGallery/Controls/SampleCodePresenter.xaml.cs
@@ -55,6 +55,8 @@ public IList<ControlExampleSubstitution> Substitutions
             set { SetValue(SubstitutionsProperty, value); }
         }
 
+        public bool IsEmpty => Code.Length == 0 && CodeSourceFile == null;
+
         private string actualCode = "";
         private static Regex SubstitutionPattern = new Regex(@"\$\(([^\)]+)\)");