Improved desktop entry menu

Currently, desktop entries display a generic menu with items that aren't
relevant to apps. This patch improves the menu by removing the unneeded
items and listing desktop specific entries such as icons.

Should compose well with pop-os/cosmic-applibrary#179

Author: joshuamegnauth54

src/app.rs

@@ -97,6 +97,7 @@ pub enum Action {
     EditHistory,
     EditLocation,
     EmptyTrash,
+    ExecEntryAction(usize),
     ExtractHere,
     Gallery,
     HistoryNext,
@@ -158,6 +159,9 @@ impl Action {
             }
             Action::EmptyTrash => Message::TabMessage(None, tab::Message::EmptyTrash),
             Action::ExtractHere => Message::ExtractHere(entity_opt),
+            Action::ExecEntryAction(&action) => {
+                Message::TabMessage(entity_opt, tab::Message::ExecEntryAction(action))
+            }
             Action::Gallery => Message::TabMessage(entity_opt, tab::Message::GalleryToggle),
             Action::HistoryNext => Message::TabMessage(entity_opt, tab::Message::GoNext),
             Action::HistoryPrevious => Message::TabMessage(entity_opt, tab::Message::GoPrevious),

src/menu.rs

@@ -86,6 +86,7 @@ pub fn context_menu<'a>(
     let mut selected_dir = 0;
     let mut selected = 0;
     let mut selected_trash_only = false;
+    let mut selected_desktop_entry = None;
     let mut selected_types: Vec<Mime> = vec![];
     tab.items_opt().map(|items| {
         for item in items.iter() {
@@ -94,8 +95,16 @@ pub fn context_menu<'a>(
                 if item.metadata.is_dir() {
                     selected_dir += 1;
                 }
-                if item.location_opt == Some(Location::Trash) {
-                    selected_trash_only = true;
+                match &item.location_opt {
+                    Some(Location::Trash) => selected_trash_only = true,
+                    Some(Location::Path(path)) => {
+                        if selected == 1
+                            && path.extension().and_then(|s| s.to_str()) == Some("desktop")
+                        {
+                            selected_desktop_entry = Some(&**path);
+                        }
+                    }
+                    _ => (),
                 }
                 selected_types.push(item.mime.clone());
             }
@@ -104,6 +113,15 @@ pub fn context_menu<'a>(
     selected_types.sort_unstable();
     selected_types.dedup();
     selected_trash_only = selected_trash_only && selected == 1;
+    // Recheck that only one item, the desktop entry, is selected
+    let selected_desktop_entry = selected_desktop_entry.and_then(|path| {
+        if selected == 1 {
+            // Cache?
+            cosmic::desktop::load_desktop_file(None, path)
+        } else {
+            None
+        }
+    });
 
     let mut children: Vec<Element<_>> = Vec::new();
     match (&tab.mode, &tab.location) {
@@ -116,6 +134,15 @@ pub fn context_menu<'a>(
                 if tab::trash_entries() > 0 {
                     children.push(menu_item(fl!("empty-trash"), Action::EmptyTrash).into());
                 }
+            } else if let Some(entry) = selected_desktop_entry {
+                children.push(menu_item(fl!("open"), Action::Open).into());
+                for (i, action) in entry.desktop_actions.into_iter().enumerate() {
+                    children.push(menu_item(action.name, Action::ExecEntryAction(i)).into())
+                }
+                children.push(divider::horizontal::light().into());
+                children.push(menu_item(fl!("rename"), Action::Rename).into());
+                // Should this simply bypass trash and remove the shortcut?
+                children.push(menu_item(fl!("move-to-trash"), Action::MoveToTrash).into());
             } else if selected > 0 {
                 if selected_dir == 1 && selected == 1 || selected_dir == 0 {
                     children.push(menu_item(fl!("open"), Action::Open).into());

src/tab.rs

@@ -1034,6 +1034,7 @@ pub enum Message {
     EditLocationEnable,
     OpenInNewTab(PathBuf),
     EmptyTrash,
+    ExecEntryAction(usize),
     Gallery(bool),
     GalleryPrevious,
     GalleryNext,
@@ -2312,6 +2313,9 @@ impl Tab {
             Message::EmptyTrash => {
                 commands.push(Command::EmptyTrash);
             }
+            Message::ExecEntryAction(action) => {
+                commands.push(Command::ExecEntryAction(action));
+            }
             Message::Gallery(gallery) => {
                 self.gallery = gallery;
             }