Code Quality: Fixed Shelf context menu

Author: yaira2

src/Files.App/UserControls/Pane/ShelfPane.xaml

@@ -105,6 +105,7 @@
 			DragItemsStarting="ListView_DragItemsStarting"
 			ItemContainerTransitions="{x:Null}"
 			ItemsSource="{x:Bind ItemsSource, Mode=OneWay}"
+			RightTapped="ShelfItemsList_RightTapped"
 			ScrollViewer.VerticalScrollBarVisibility="Auto"
 			ScrollViewer.VerticalScrollMode="Auto"
 			SelectionMode="Extended">
@@ -126,16 +127,6 @@
 							TextTrimming="CharacterEllipsis"
 							TextWrapping="NoWrap"
 							ToolTipService.ToolTip="{x:Bind Path, Mode=OneWay}" />
-
-						<Grid.ContextFlyout>
-							<MenuFlyout>
-								<MenuFlyoutItem Command="{x:Bind RemoveCommand}" Text="{helpers:ResourceString Name=RemoveFromShelf}">
-									<MenuFlyoutItem.Icon>
-										<FontIcon Glyph="&#xE738;" />
-									</MenuFlyoutItem.Icon>
-								</MenuFlyoutItem>
-							</MenuFlyout>
-						</Grid.ContextFlyout>
 					</Grid>
 				</DataTemplate>
 			</ListView.ItemTemplate>

src/Files.App/UserControls/Pane/ShelfPane.xaml.cs

@@ -86,6 +86,26 @@ private void ListView_DragItemsStarting(object sender, DragItemsStartingEventArg
 			dataObjectProvider.SetDataObject(ppDataObject);
 		}
 
+		private void ShelfItemsList_RightTapped(object sender, Microsoft.UI.Xaml.Input.RightTappedRoutedEventArgs e)
+		{
+			if (e.OriginalSource is not Microsoft.UI.Xaml.FrameworkElement widgetCardItem ||
+				widgetCardItem.DataContext is not ShelfItem item ||
+				item.Path is null)
+				return;
+
+			var menuFlyout = new MenuFlyout();
+
+			menuFlyout.Items.Add (new MenuFlyoutItem
+			{
+				Text = Strings.RemoveFromShelf.GetLocalizedResource(),
+				Icon = new FontIcon { Glyph = "\uE738" },
+				Command = new RelayCommand(item.Remove)
+			});
+
+			menuFlyout.ShowAt(widgetCardItem);
+			e.Handled = true;
+		}
+
 		public ObservableCollection<ShelfItem>? ItemsSource
 		{
 			get => (ObservableCollection<ShelfItem>?)GetValue(ItemsSourceProperty);