Added comments

Author: yaira2

src/Files.App/UserControls/TabBar/TabBar.xaml.cs

@@ -63,8 +63,11 @@ public Rectangle DragArea
 		/// <summary> Starting time when dragging a tab. </summary>
 		private DateTimeOffset dragStartTime;
 
-		/// <summary> Cancel the tag drag event. </summary>
-		private bool cancelTagDrag;
+		/// <summary>
+		/// Indicates if drag operation should be canceled.
+		/// This value gets reset at the start of the drag operation
+		/// </summary>
+		private bool isCancelingDragOperation;
 
 		public TabBar()
 		{
@@ -86,9 +89,7 @@ public TabBar()
 		private void TabView_TabItemsChanged(TabView sender, Windows.Foundation.Collections.IVectorChangedEventArgs args)
 		{
 			if (args.CollectionChange == Windows.Foundation.Collections.CollectionChange.ItemRemoved)
-			{
 				App.AppModel.TabStripSelectedIndex = Items.IndexOf(HorizontalTabView.SelectedItem as TabBarItem);
-			}
 
 			if (App.AppModel.TabStripSelectedIndex >= 0 && App.AppModel.TabStripSelectedIndex < Items.Count)
 			{
@@ -136,30 +137,34 @@ private void TabHoverSelected(object sender, object e)
 		{
 			tabHoverTimer.Stop();
 			if (hoveredTabViewItem is not null)
-			{
 				App.AppModel.TabStripSelectedIndex = Items.IndexOf(hoveredTabViewItem.DataContext as TabBarItem);
-			}
 		}
 
 		private void TabView_TabDragStarting(TabView sender, TabViewTabDragStartingEventArgs args)
 		{
+			// Reset value
+			isCancelingDragOperation = false;
+
 			var tabViewItemArgs = (args.Item as TabBarItem).NavigationParameter;
 			args.Data.Properties.Add(TabPathIdentifier, tabViewItemArgs.Serialize());
 			args.Data.RequestedOperation = DataPackageOperation.Move;
 
+			// Get cursor position & time to track how far the tab was dragged.
 			InteropHelpers.GetCursorPos(out dragStartPoint);
 			dragStartTime = DateTimeOffset.UtcNow;
 
-			// Force focus in order that the keydown event works
+			// Focus the UI Element, without this the focus sometimes changes
+			// and the PreviewKeyDown event won't trigger.
 			Focus(FocusState.Programmatic);
-			cancelTagDrag = false;
-			PreviewKeyDown += DraggingKeyDown;
+			PreviewKeyDown += TabDragging_PreviewKeyDown;
 		}
 
-		private void DraggingKeyDown(object sender, KeyRoutedEventArgs e)
+		private void TabDragging_PreviewKeyDown(object sender, KeyRoutedEventArgs e)
 		{
+			// Pressing escape will automatically complete the drag event but we need to set the
+			// isCancelingDragOperation field in order to detect if escape was pressed.
 			if (e.Key is Windows.System.VirtualKey.Escape)
-				cancelTagDrag = true;
+				isCancelingDragOperation = true;
 		}
 
 		private void TabView_TabStripDragOver(object sender, DragEventArgs e)
@@ -193,9 +198,7 @@ private async void TabView_TabStripDrop(object sender, DragEventArgs e)
 
 			if (!e.DataView.Properties.TryGetValue(TabPathIdentifier, out object tabViewItemPathObj) ||
 				!(tabViewItemPathObj is string tabViewItemString))
-			{
 				return;
-			}
 
 			var index = -1;
 
@@ -217,29 +220,25 @@ private async void TabView_TabStripDrop(object sender, DragEventArgs e)
 
 		private void TabView_TabDragCompleted(TabView sender, TabViewTabDragCompletedEventArgs args)
 		{
+			// Unsubscribe from the key down event, it's only needed when a tag is actively being dragged
+			PreviewKeyDown -= TabDragging_PreviewKeyDown;
+
 			if (ApplicationData.Current.LocalSettings.Values.ContainsKey(TabDropHandledIdentifier) &&
 				(bool)ApplicationData.Current.LocalSettings.Values[TabDropHandledIdentifier])
-			{
 				CloseTab(args.Item as TabBarItem);
-			}
 			else
-			{
 				HorizontalTabView.SelectedItem = args.Tab;
-			}
 
 			if (ApplicationData.Current.LocalSettings.Values.ContainsKey(TabDropHandledIdentifier))
-			{
 				ApplicationData.Current.LocalSettings.Values.Remove(TabDropHandledIdentifier);
-			}
-
-			PreviewKeyDown -= DraggingKeyDown;
 		}
 
 		private async void TabView_TabDroppedOutside(TabView sender, TabViewTabDroppedOutsideEventArgs args)
 		{
-			PreviewKeyDown -= DraggingKeyDown;
+			// Unsubscribe from the key down event, it's only needed when a tag is actively being dragged
+			PreviewKeyDown -= TabDragging_PreviewKeyDown;
 
-			if (cancelTagDrag)
+			if (isCancelingDragOperation)
 				return;
 
 			InteropHelpers.GetCursorPos(out var droppedPoint);
@@ -263,10 +262,8 @@ private async void TabView_TabDroppedOutside(TabView sender, TabViewTabDroppedOu
 				sender.SelectedIndex = selectedTabViewItemIndex;
 			}
 			else
-			{
 				// Dispose tab arguments
 				(args.Item as TabBarItem)?.Unload();
-			}
 		}
 
 		private void TabItemContextMenu_Opening(object sender, object e)