Add option to hide TimeStamp columns in EventWindow View menu (#2247)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: brianrob <6210322+brianrob@users.noreply.github.com>

Author: Copilot

src/PerfView/EventViewer/EventWindow.xaml

@@ -87,7 +87,8 @@
                     <MenuItem Header="_Close" Click="DoClose"/>
                 </MenuItem>
                 <MenuItem Header="_View">
-                    <MenuItem Header="_Show Local Time" IsCheckable="True" IsChecked="False" Checked="DoUseLocalTime" Unchecked="DoUseOriginTime"/>
+                    <MenuItem x:Name="ShowTimeStampColumnsMenuItem" Header="Show _TimeStamp Columns" IsCheckable="True" IsChecked="True" Checked="DoShowTimeStampColumns" Unchecked="DoHideTimeStampColumns"/>
+                    <MenuItem x:Name="ShowLocalTimeMenuItem" Header="_Show Local Time" IsCheckable="True" IsChecked="False" Checked="DoUseLocalTime" Unchecked="DoUseOriginTime"/>
                 </MenuItem>
                 <MenuItem Header="_Help">
                     <MenuItem Header="_Help on Event Viewer" Command="Help" CommandParameter="EventViewerQuickStart" />

src/PerfView/EventViewer/EventWindow.xaml.cs

@@ -57,6 +57,12 @@ public EventWindow(EventWindow template)
             {
                 selection.Add(item);
             }
+            
+            // Copy timestamp column visibility settings from template
+            ShowTimeStampColumnsMenuItem.IsChecked = template.ShowTimeStampColumnsMenuItem.IsChecked;
+            ShowLocalTimeMenuItem.IsChecked = template.ShowLocalTimeMenuItem.IsChecked;
+            ShowLocalTimeMenuItem.IsEnabled = template.ShowLocalTimeMenuItem.IsEnabled;
+            
             Update();
         }
         public EventWindow(Window parent, PerfViewEventSource data)
@@ -167,6 +173,26 @@ public EventWindow(Window parent, PerfViewEventSource data)
             };
 
             MultiLineViewPaneHidden = (App.UserConfigData["MultiLineViewPaneHidden"] == "true");
+            
+            // Initialize timestamp column visibility based on user preference
+            bool showTimeStampColumns = App.UserConfigData["EventWindowShowTimeStampColumns"] != "false"; // Default to true
+            ShowTimeStampColumnsMenuItem.IsChecked = showTimeStampColumns;
+            if (!showTimeStampColumns)
+            {
+                // Hide both timestamp columns and disable the timezone menu
+                foreach (var column in Grid.Columns)
+                {
+                    if (column == OriginTimeStampColumn || column == LocalTimeStampColumn)
+                    {
+                        column.Visibility = Visibility.Hidden;
+                    }
+                }
+                ShowLocalTimeMenuItem.IsEnabled = false;
+            }
+            else
+            {
+                ShowLocalTimeMenuItem.IsEnabled = true;
+            }
         }
 
         public PerfViewEventSource DataSource { get; private set; }
@@ -1893,32 +1919,87 @@ private void Add(ObservableCollection<EventRecord> events, EventRecord event_)
 
         private void DoUseLocalTime(object sender, RoutedEventArgs e)
         {
-            foreach (var i in Grid.Columns)
+            // Only change visibility if timestamp columns are enabled
+            if (ShowTimeStampColumnsMenuItem.IsChecked)
             {
-                if (i == OriginTimeStampColumn)
+                foreach (var i in Grid.Columns)
                 {
-                    i.Visibility = Visibility.Hidden;
+                    if (i == OriginTimeStampColumn)
+                    {
+                        i.Visibility = Visibility.Hidden;
+                    }
+                    else if (i == LocalTimeStampColumn)
+                    {
+                        i.Visibility = Visibility.Visible;
+                    }
                 }
-                else if (i == LocalTimeStampColumn)
+            }
+        }
+
+        private void DoUseOriginTime(object sender, RoutedEventArgs e)
+        {
+            // Only change visibility if timestamp columns are enabled
+            if (ShowTimeStampColumnsMenuItem.IsChecked)
+            {
+                foreach (var i in Grid.Columns)
                 {
-                    i.Visibility = Visibility.Visible;
+                    if (i == OriginTimeStampColumn)
+                    {
+                        i.Visibility = Visibility.Visible;
+                    }
+                    else if (i == LocalTimeStampColumn)
+                    {
+                        i.Visibility = Visibility.Hidden;
+                    }
                 }
             }
         }
 
-        private void DoUseOriginTime(object sender, RoutedEventArgs e)
+        private void DoShowTimeStampColumns(object sender, RoutedEventArgs e)
         {
+            // Check if UI elements are initialized to avoid null reference during XAML construction
+            if (ShowLocalTimeMenuItem == null || Grid?.Columns == null)
+                return;
+                
+            // Show the appropriate timestamp column based on current preference
+            bool useLocalTime = ShowLocalTimeMenuItem.IsChecked;
             foreach (var i in Grid.Columns)
             {
                 if (i == OriginTimeStampColumn)
                 {
-                    i.Visibility = Visibility.Visible;
+                    i.Visibility = useLocalTime ? Visibility.Hidden : Visibility.Visible;
                 }
                 else if (i == LocalTimeStampColumn)
+                {
+                    i.Visibility = useLocalTime ? Visibility.Visible : Visibility.Hidden;
+                }
+            }
+            // Enable the Show Local Time menu item
+            ShowLocalTimeMenuItem.IsEnabled = true;
+            
+            // Save preference
+            App.UserConfigData["EventWindowShowTimeStampColumns"] = "true";
+        }
+
+        private void DoHideTimeStampColumns(object sender, RoutedEventArgs e)
+        {
+            // Check if UI elements are initialized to avoid null reference during XAML construction
+            if (ShowLocalTimeMenuItem == null || Grid?.Columns == null)
+                return;
+                
+            // Hide both timestamp columns
+            foreach (var i in Grid.Columns)
+            {
+                if (i == OriginTimeStampColumn || i == LocalTimeStampColumn)
                 {
                     i.Visibility = Visibility.Hidden;
                 }
             }
+            // Gray out (disable) the Show Local Time menu item
+            ShowLocalTimeMenuItem.IsEnabled = false;
+            
+            // Save preference
+            App.UserConfigData["EventWindowShowTimeStampColumns"] = "false";
         }
     }
 }