From 400672b6b6453fdbe876d9828f2256318c1eb7a9 Mon Sep 17 00:00:00 2001
From: Christopher Davis <christopherdavis@gnome.org>
Date: Tue, 28 Dec 2021 16:38:36 -0800
Subject: [PATCH] window: Add ToastOverlay and use toasts for feedback

---
 data/gtk/window.ui        | 52 +++++++++++++++++++++------------------
 src/widgets/image_view.rs | 40 ++++++++++++++++--------------
 src/window.rs             | 30 +++++++++++++++++++---
 3 files changed, 76 insertions(+), 46 deletions(-)

diff --git a/data/gtk/window.ui b/data/gtk/window.ui
index 510d42f..7feb7f5 100644
--- a/data/gtk/window.ui
+++ b/data/gtk/window.ui
@@ -38,34 +38,38 @@
           </object>
         </property>
         <property name="content">
-          <object class="GtkStack" id="stack">
-            <property name="visible-child">status_page</property>
-            <property name="vexpand">True</property>
-            <child>
-              <object class="AdwStatusPage" id="status_page">
-                <property name="title" translatable="yes">Image Viewer</property>
-                <property name="description" translatable="yes">Click "Open File…" or drag an image here.</property>
+          <object class="AdwToastOverlay" id="toast_overlay">
+            <property name="child">
+              <object class="GtkStack" id="stack">
+                <property name="visible-child">status_page</property>
+                <property name="vexpand">True</property>
                 <child>
-                  <object class="GtkButton">
-                    <property name="halign">center</property>
-                    <property name="label" translatable="yes">Open File…</property>
-                    <property name="action-name">win.open</property>
-                    <style>
-                      <class name="pill"/>
-                      <class name="suggested-action"/>
-                    </style>
+                  <object class="AdwStatusPage" id="status_page">
+                    <property name="title" translatable="yes">Image Viewer</property>
+                    <property name="description" translatable="yes">Click "Open File…" or drag an image here.</property>
+                    <child>
+                      <object class="GtkButton">
+                        <property name="halign">center</property>
+                        <property name="label" translatable="yes">Open File…</property>
+                        <property name="action-name">win.open</property>
+                        <style>
+                          <class name="pill"/>
+                          <class name="suggested-action"/>
+                        </style>
+                      </object>
+                    </child>
+                  </object>
+                </child>
+                <child>
+                  <object class="LpImageView" id="image_view"/>
+                </child>
+                <child>
+                  <object class="GtkDropTarget" id="drop_target">
+                    <property name="actions">copy</property>
                   </object>
                 </child>
               </object>
-            </child>
-            <child>
-              <object class="LpImageView" id="image_view"/>
-            </child>
-            <child>
-              <object class="GtkDropTarget" id="drop_target">
-                <property name="actions">copy</property>
-              </object>
-            </child>
+            </property>
           </object>
         </property>
         <style>
diff --git a/src/widgets/image_view.rs b/src/widgets/image_view.rs
index 7909bb3..5b8036b 100644
--- a/src/widgets/image_view.rs
+++ b/src/widgets/image_view.rs
@@ -271,9 +271,27 @@ impl LpImageView {
     pub fn set_wallpaper(&self) -> anyhow::Result<()> {
         let wallpaper = self.uri().context("No URI for current file")?;
         let ctx = glib::MainContext::default();
-        ctx.spawn_local(async move {
-            set_wallpaper(wallpaper).await;
-        });
+        ctx.spawn_local(clone!(@weak self as view => async move {
+            let status = match wallpaper::set_from_uri(
+                &WindowIdentifier::default(),
+                &wallpaper,
+                true,
+                wallpaper::SetOn::Background,
+            )
+            .await {
+                Ok(_) => "Set as wallpaper.",
+                Err(_) => "Could not set wallpaper.",
+            };
+
+            view.activate_action(
+                "win.show-toast",
+                // We use `1` here because we can't pass enums directly as GVariants,
+                // so we need to use the C int value of the enum.
+                // `TOAST_PRIORITY_NORMAL = 0`, and `TOAST_PRIORITY_HIGH = 1`
+                Some(&(status, 1).to_variant()),
+            )
+            .unwrap();
+        }));
 
         Ok(())
     }
@@ -338,19 +356,3 @@ impl LpImageView {
         imp.popover.popup();
     }
 }
-
-async fn set_wallpaper(uri: String) {
-    if let Err(e) = wallpaper::set_from_uri(
-        &WindowIdentifier::default(),
-        &uri,
-        true,
-        wallpaper::SetOn::Background,
-    )
-    .await
-    {
-        log::error!(
-            "Failed to set the wallpaper using the freedesktop portal: {}",
-            e
-        );
-    }
-}
diff --git a/src/window.rs b/src/window.rs
index 7187c89..11c7024 100644
--- a/src/window.rs
+++ b/src/window.rs
@@ -61,6 +61,8 @@ mod imp {
         #[template_child]
         pub fullscreen_button: TemplateChild<gtk::Button>,
         #[template_child]
+        pub toast_overlay: TemplateChild<adw::ToastOverlay>,
+        #[template_child]
         pub stack: TemplateChild<gtk::Stack>,
         #[template_child]
         pub status_page: TemplateChild<adw::StatusPage>,
@@ -101,6 +103,12 @@ mod imp {
             klass.install_action("win.print", None, move |win, _, _| {
                 win.print();
             });
+
+            klass.install_action("win.show-toast", Some("(si)"), move |win, _, var| {
+                if let Some((ref toast, i)) = var.map(|v| v.get::<(String, i32)>()).flatten() {
+                    win.show_toast(toast, adw::ToastPriority::__Unknown(i));
+                }
+            });
         }
 
         fn instance_init(obj: &glib::subclass::InitializingObject<Self>) {
@@ -155,8 +163,12 @@ mod imp {
                     let file = list.files().get(0).unwrap().clone();
                     let info = util::query_attributes(
                         &file,
-                        vec![&gio::FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE])
-                        .expect("Could not query file info");
+                        vec![
+                            &gio::FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE,
+                            &gio::FILE_ATTRIBUTE_STANDARD_DISPLAY_NAME,
+                        ],
+                    )
+                    .expect("Could not query file info");
 
                     if info
                         .content_type()
@@ -165,7 +177,10 @@ mod imp {
                         .is_some() {
                         obj.set_image_from_file(&file, false);
                     } else {
-                        log::debug!("{} was not a valid image.", file.uri().to_string());
+                        obj.show_toast(
+                            &format!("\"{}\" is not a valid image.", info.display_name().to_string()),
+                            adw::ToastPriority::High,
+                        );
                     }
 
                     true
@@ -275,6 +290,15 @@ impl LpWindow {
         }
     }
 
+    fn show_toast(&self, text: &impl AsRef<str>, priority: adw::ToastPriority) {
+        let imp = imp::LpWindow::from_instance(self);
+
+        let toast = adw::Toast::new(text.as_ref());
+        toast.set_priority(priority);
+
+        imp.toast_overlay.add_toast(&toast);
+    }
+
     pub fn set_image_from_file(&self, file: &gio::File, resize: bool) {
         let imp = imp::LpWindow::from_instance(self);