Add button to open path in media modal

Author: mtkennerly

src/gui/app.rs

@@ -762,7 +762,7 @@ impl App {
                     match result {
                         Ok(_) => Message::Ignore,
                         Err(e) => {
-                            log::error!("Unable to open directory: `{}` - {:?}", path2.resolve(), e);
+                            log::error!("Unable to open file: `{}` - {:?}", path2.resolve(), e);
                             Message::OpenPathFailure { path: path2 }
                         }
                     }

src/gui/button.rs

@@ -232,6 +232,20 @@ pub fn choose_file<'a>(
     ))
 }
 
+pub fn open_path<'a>(path: StrictPath, modifiers: &keyboard::Modifiers) -> CustomButton<'a> {
+    if modifiers.shift() {
+        icon(Icon::FileOpen).on_press(Message::OpenFile { path })
+    } else {
+        icon(Icon::OpenInNew).on_press(Message::OpenDir { path })
+    }
+    .tooltip(format!(
+        "{}\n{} {}",
+        lang::action::open_folder(),
+        lang::field(&lang::thing::key::shift()),
+        lang::action::open_file()
+    ))
+}
+
 pub fn move_up<'a>(action: fn(EditAction) -> Message, index: usize) -> CustomButton<'a> {
     icon(Icon::ArrowUpward).on_press_maybe((index > 0).then(|| action(EditAction::move_up(index))))
 }

src/gui/modal.rs

@@ -457,7 +457,7 @@ impl Modal {
                 for media in all_media {
                     col = col.push(
                         Row::new()
-                            .spacing(20)
+                            .spacing(10)
                             .align_y(Alignment::Center)
                             .push(if collection.is_error(media) {
                                 button::icon(Icon::Error)
@@ -478,7 +478,8 @@ impl Modal {
                                 }
                                 .small_control(),
                             )
-                            .push(text(media.path().raw()).width(Length::Fill)),
+                            .push(button::open_path(media.path().clone(), modifiers))
+                            .push(text(media.path().raw())),
                     );
                 }
             }