Cleanup unnecessary code (#1299)

Author: marcelwgn

WinUIGallery/App.xaml.cs

@@ -116,11 +116,6 @@ protected override void OnLaunched(Microsoft.UI.Xaml.LaunchActivatedEventArgs ar
             startupWindow = WindowHelper.CreateWindow();
             startupWindow.ExtendsContentIntoTitleBar = true;
 #if DEBUG
-            //if (System.Diagnostics.Debugger.IsAttached)
-            //{
-            //    this.DebugSettings.EnableFrameRateCounter = true;
-            //}
-
             if (System.Diagnostics.Debugger.IsAttached)
             {
                 this.DebugSettings.BindingFailed += DebugSettings_BindingFailed;

WinUIGallery/Common/ActivityFeedLayout.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using Windows.Foundation;
 using Microsoft.UI.Xaml;
@@ -95,8 +95,7 @@ private static void OnPropertyChanged(DependencyObject obj, DependencyPropertyCh
         protected override void InitializeForContextCore(VirtualizingLayoutContext context)
         {
             base.InitializeForContextCore(context);
-
-            if (!(context.LayoutState is ActivityFeedLayoutState state))
+            if (!(context.LayoutState is ActivityFeedLayoutState))
             {
                 // Store any state we might need since (in theory) the layout could be in use by multiple
                 // elements simultaneously
@@ -169,7 +168,6 @@ protected override Size MeasureOverride(VirtualizingLayoutContext context, Size
                 for (int columnIndex = 0; columnIndex < 3; columnIndex++)
                 {
                     var index = firstItemIndex + columnIndex;
-                    var rect = boundsForCurrentRow[index % 3];
                     var container = context.GetOrCreateElementAt(index);
 
                     container.Measure(

WinUIGallery/Common/Category.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using Microsoft.UI.Xaml.Controls;
 
 namespace AppUIBasics.Common
@@ -11,7 +11,6 @@ public class Category : CategoryBase
         public string Name { get; set; }
         public string Tooltip { get; set; }
         public Symbol Glyph { get; set; }
-        //public Type TargetType { get; set; }
     }
 
     public class Separator : CategoryBase { }

WinUIGallery/Common/ColorSlideTransitionHelper.cs

@@ -1,4 +1,4 @@
-//*********************************************************
+//*********************************************************
 //
 // Copyright (c) Microsoft. All rights reserved.
 // This code is licensed under the MIT License (MIT).
@@ -25,7 +25,6 @@ public class ColorSlideTransitionHelper
     {
         #region Member variables
 
-        UIElement hostForVisual;
         Compositor _compositor;
         ContainerVisual _containerForVisuals;
         ScalarKeyFrameAnimation _slideAnimation;
@@ -42,10 +41,6 @@ public class ColorSlideTransitionHelper
         /// </summary>
         public ColorSlideTransitionHelper(UIElement hostForVisual)
         {
-
-
-            this.hostForVisual = hostForVisual;
-
             // we have an element in the XAML tree that will host our Visuals
             var visual = ElementCompositionPreview.GetElementVisual(hostForVisual);
             _compositor = visual.Compositor;

WinUIGallery/Common/MenuItemTemplateSelector.cs

@@ -1,4 +1,4 @@
-using Microsoft.UI.Xaml;
+using Microsoft.UI.Xaml;
 using Microsoft.UI.Xaml.Controls;
 using Microsoft.UI.Xaml.Markup;
 using System;
@@ -11,8 +11,6 @@ class MenuItemTemplateSelector : DataTemplateSelector
     {
         public DataTemplate ItemTemplate { get; set; }
 
-        //public string PaneTitle { get; set; }
-
         protected override DataTemplate SelectTemplateCore(object item)
         {
             return item is Separator ? SeparatorTemplate : item is Header ? HeaderTemplate : ItemTemplate;

WinUIGallery/Common/Win32.cs

@@ -19,7 +19,6 @@ internal static class Win32
 
         public const int WM_ACTIVATE = 0x0006;
         public const int WA_ACTIVE = 0x01;
-        //static int WA_CLICKACTIVE = 0x02;
         public const int WA_INACTIVE = 0x00;
 
         public const int WM_SETICON = 0x0080;

WinUIGallery/Common/WrapPanel.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Diagnostics.CodeAnalysis;
 using System.Globalization;
@@ -38,7 +38,6 @@ public class WrapPanel : Panel
         /// within a <see cref="T:WinRTXamlToolkit.Controls.WrapPanel" />.  The
         /// default value is <see cref="F:System.Double.NaN" />.
         /// </value>
-        //[TypeConverter(typeof(LengthConverter))]
         public double ItemHeight
         {
             get { return (double)GetValue(ItemHeightProperty); }
@@ -73,7 +72,6 @@ public double ItemHeight
         /// contained in a <see cref="T:WinRTXamlToolkit.Controls.WrapPanel" />.
         /// The default value is <see cref="F:System.Double.NaN" />.
         /// </value>
-        //[TypeConverter(typeof(LengthConverter))]
         public double ItemWidth
         {
             get { return (double)GetValue(ItemWidthProperty); }

WinUIGallery/ConnectedAnimationPages/CollectionPage.xaml.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using Microsoft.UI.Xaml;
 using Microsoft.UI.Xaml.Controls;
 using Microsoft.UI.Xaml.Media.Animation;
@@ -59,8 +59,7 @@ private void collection_ItemClick(object sender, ItemClickEventArgs e)
                 // Prepare the connected animation.
                 // Notice that the stored item is passed in, as well as the name of the connected element. 
                 // The animation will actually start on the Detailed info page.
-                var animation = collection.PrepareConnectedAnimation("ForwardConnectedAnimation", _storeditem, "connectedElement");
-
+                collection.PrepareConnectedAnimation("ForwardConnectedAnimation", _storeditem, "connectedElement");
             }
 
             // Navigate to the DetailedInfoPage.

WinUIGallery/ControlPages/AppBarSeparatorPage.xaml.cs

@@ -20,16 +20,5 @@ public AppBarSeparatorPage()
         {
             this.InitializeComponent();
         }
-        private void CompactButton_Click(object sender, RoutedEventArgs e)
-        {
-            if ((sender as AppBarToggleButton).IsChecked == true)
-            {
-                Control1.DefaultLabelPosition = CommandBarDefaultLabelPosition.Collapsed;
-            }
-            else
-            {
-                Control1.DefaultLabelPosition = CommandBarDefaultLabelPosition.Bottom;
-            }
-        }
     }
 }

WinUIGallery/ControlPages/AppBarToggleButtonPage.xaml.cs

@@ -21,17 +21,6 @@ public AppBarToggleButtonPage()
             this.InitializeComponent();
         }
 
-        private void CompactButton_Click(object sender, RoutedEventArgs e)
-        {
-            if (sender is ToggleButton toggle && toggle.IsChecked != null)
-            {
-                Button1.IsCompact =
-                Button2.IsCompact =
-                Button3.IsCompact =
-                Button4.IsCompact = (bool)toggle.IsChecked;
-            }
-        }
-
         private void AppBarButton_Click(object sender, RoutedEventArgs e)
         {
             if (sender is AppBarToggleButton b)