Added a right click context menu to the drives widget (#2343)

Co-authored-by: Marco Gavelli <[email protected]>

Author: yaira2

Files/Filesystem/DriveItem.cs

@@ -22,6 +22,7 @@ public class DriveItem : ObservableObject, INavigationControlItem
         public ByteSize FreeSpace { get; set; }
         public ByteSize SpaceUsed { get; set; }
         public Visibility ItemVisibility { get; set; } = Visibility.Visible;
+        public bool IsRemovable { get; set; }
 
         private DriveType type;
 
@@ -62,6 +63,7 @@ public DriveItem(StorageFolder root, DriveType type)
             Type = type;
             Path = string.IsNullOrEmpty(root.Path) ? $"\\\\?\\{root.Name}\\" : root.Path;
             Root = root;
+            IsRemovable = (Type == DriveType.Removable || Type == DriveType.CDRom);
 
             CoreApplication.MainView.ExecuteOnUIThreadAsync(() => GetDriveItemProperties());
         }

Files/UserControls/SidebarControl.xaml.cs

@@ -241,15 +241,7 @@ private void NavigationViewDriveItem_RightTapped(object sender, RightTappedRoute
             Microsoft.UI.Xaml.Controls.NavigationViewItem sidebarItem = (Microsoft.UI.Xaml.Controls.NavigationViewItem)sender;
             var item = sidebarItem.DataContext as DriveItem;
 
-            if (item.Type == DriveType.Removable || item.Type == DriveType.CDRom)
-            {
-                ShowEjectDevice = true;
-            }
-            else
-            {
-                ShowEjectDevice = false;
-            }
-
+            ShowEjectDevice = item.IsRemovable;
             ShowUnpinItem = false;
             ShowEmptyRecycleBin = false;
             ShowProperties = true;

Files/UserControls/Widgets/DrivesWidget.xaml

@@ -75,6 +75,43 @@
                             Style="{StaticResource ButtonRevealStyle}"
                             Tag="{x:Bind Path}"
                             ToolTipService.ToolTip="{x:Bind Text, Mode=OneWay}">
+                            <Button.ContextFlyout>
+                                <MenuFlyout>
+                                    <MenuFlyout.Items>
+                                        <MenuFlyoutItem
+                                            x:Name="OpenInNewTab"
+                                            x:Uid="SideBarOpenInNewTab"
+                                            Click="OpenInNewTab_Click"
+                                            DataContext="{x:Bind}"
+                                            Text="Open in new tab">
+                                            <MenuFlyoutItem.Icon>
+                                                <FontIcon FontFamily="{StaticResource CustomGlyph}" Glyph="&#xF106;" />
+                                            </MenuFlyoutItem.Icon>
+                                        </MenuFlyoutItem>
+                                        <MenuFlyoutItem
+                                            x:Name="OpenInNewWindow"
+                                            x:Uid="SideBarOpenInNewWindow"
+                                            Click="OpenInNewWindow_Click"
+                                            DataContext="{x:Bind}"
+                                            Text="Open in new window">
+                                            <MenuFlyoutItem.Icon>
+                                                <FontIcon FontFamily="{StaticResource CustomGlyph}" Glyph="&#xF107;" />
+                                            </MenuFlyoutItem.Icon>
+                                        </MenuFlyoutItem>
+                                        <MenuFlyoutItem
+                                            x:Name="EjectDevice"
+                                            x:Uid="SideBarEjectDevice"
+                                            x:Load="{x:Bind IsRemovable, Mode=OneWay}"
+                                            Click="EjectDevice_Click"
+                                            DataContext="{x:Bind}"
+                                            Text="Eject">
+                                            <MenuFlyoutItem.Icon>
+                                                <FontIcon FontFamily="{StaticResource CustomGlyph}" Glyph="&#xF10B;" />
+                                            </MenuFlyoutItem.Icon>
+                                        </MenuFlyoutItem>
+                                    </MenuFlyout.Items>
+                                </MenuFlyout>
+                            </Button.ContextFlyout>
                             <Grid
                                 Margin="12"
                                 HorizontalAlignment="Stretch"

Files/UserControls/Widgets/DrivesWidget.xaml.cs

@@ -3,9 +3,11 @@
 using System;
 using System.Collections.ObjectModel;
 using System.Numerics;
+using Files.Interacts;
 using Windows.UI.Xaml;
 using Windows.UI.Xaml.Controls;
 using Windows.UI.Xaml.Hosting;
+using System.Linq;
 
 namespace Files
 {
@@ -24,6 +26,24 @@ public DrivesWidget()
             InitializeComponent();
         }
 
+        private async void EjectDevice_Click(object sender, RoutedEventArgs e)
+        {
+            var item = ((MenuFlyoutItem)sender).DataContext as DriveItem;
+            await Interaction.EjectDeviceAsync(item.Path);
+        }
+
+        private void OpenInNewTab_Click(object sender, RoutedEventArgs e)
+        {
+            var item = ((MenuFlyoutItem)sender).DataContext as DriveItem;
+            Interaction.OpenPathInNewTab(item.Path);
+        }
+
+        private async void OpenInNewWindow_Click(object sender, RoutedEventArgs e)
+        {
+            var item = ((MenuFlyoutItem)sender).DataContext as DriveItem;
+            await Interaction.OpenPathInNewWindowAsync(item.Path);
+        }
+
         private void Button_Click(object sender, RoutedEventArgs e)
         {
             string NavigationPath = ""; // path to navigate