From 6ebb1c3f9d9b42f6b13b1605d7e234c85a07bc2e Mon Sep 17 00:00:00 2001
From: Feddy's Thinkpad <cavaliere12.web@gmail.com>
Date: Fri, 5 Apr 2024 18:53:10 +0200
Subject: [PATCH] store file path in tooltip

basename path is showed when listing file in the CURRENT_DIRECTORY or with the `-b` flag
---
 src/list_view.rs | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/list_view.rs b/src/list_view.rs
index 72dad8c..3e41a29 100644
--- a/src/list_view.rs
+++ b/src/list_view.rs
@@ -110,7 +110,7 @@ fn get_file(row: &CenterBox) -> gio::File {
     } else {
         row.center_widget().unwrap()
     };
-    gio::File::for_path(file_widget.downcast::<Label>().unwrap().text())
+    gio::File::for_path(file_widget.tooltip_text().unwrap())
 }
 
 // Setup the widgets in the ListView
@@ -146,8 +146,11 @@ fn setup_factory(factory: &SignalListItemFactory, list: &MultiSelection) {
             .and_downcast::<CenterBox>()
             .expect("The child has to be a `Label`.");
 
-        // show either relative or absolute path
         // This is safe because the file needs to exist
+        let path = file_object.file().parse_name().to_string();
+
+        // show either relative or absolute path
+        // only used for the display label
         let str = if ARGS.get().unwrap().basename || file_object
             .file()
             .has_parent(Some(CURRENT_DIRECTORY.get().unwrap())
@@ -157,15 +160,14 @@ fn setup_factory(factory: &SignalListItemFactory, list: &MultiSelection) {
                 .to_str().unwrap()
                 .to_string()
         } else {
-            file_object.file()
-                .parse_name().to_string()
+            path.to_owned()
         };
 
         let label = Label::builder()
             .label(&str)
             .hexpand(true)
             .ellipsize(gtk::pango::EllipsizeMode::End)
-            .tooltip_text(&str);
+            .tooltip_text(&path);
 
         if ARGS.get().unwrap().icons_only {
             file_row.set_start_widget(Some(&label.visible(false).build()));