From 811670b3728e7f2d738f21f4f613592128eb3188 Mon Sep 17 00:00:00 2001
From: Max Bradbury <max@tinybird.info>
Date: Sun, 16 Jan 2022 15:47:39 +0000
Subject: [PATCH] provide an info popup when opening sounds folder

---
 src/ui_gtk.rs | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/src/ui_gtk.rs b/src/ui_gtk.rs
index 37271d0..cfd6dda 100644
--- a/src/ui_gtk.rs
+++ b/src/ui_gtk.rs
@@ -9,9 +9,9 @@ use rodio::{Sink, Source};
 
 const SPACING: i32 = 16;
 
-fn error_popup(message: &str) {
+fn popup(title: &str, message: &str) -> gtk::Window {
     let popup = gtk::Window::new(gtk::WindowType::Toplevel);
-    popup.set_title("error");
+    popup.set_title(title);
     popup.set_border_width(SPACING as u32);
     popup.set_position(gtk::WindowPosition::Center);
     popup.set_default_size(256, 64);
@@ -21,7 +21,7 @@ fn error_popup(message: &str) {
     let message = gtk::Label::new(Some(message));
     popup.add(&message);
 
-    popup.show_all();
+    popup
 }
 
 fn build_ui(application: &gtk::Application) {
@@ -40,6 +40,14 @@ fn build_ui(application: &gtk::Application) {
     let button_manage_sounds = gtk::Button::with_label("open sounds folder");
 
     button_manage_sounds.connect_clicked(|_| {
+        popup(
+            "info",
+            concat!(
+                "after changing the sounds in this folder, ",
+                "you will need to restart lull to see your changes."
+            )
+        );
+
         let mut file_manager = Command::new(crate::file_manager());
         file_manager.arg(crate::get_sounds_dir());
         file_manager.output().unwrap();
@@ -91,11 +99,12 @@ fn build_ui(application: &gtk::Application) {
         );
 
         if source.is_err() {
-            error_popup(&format!(
+            popup("error", &format!(
                 "Couldn't parse file {}. \n{}.",
                 path.to_str().unwrap(),
                 source.err().unwrap()
-            ));
+            )).show_all();
+
             continue;
         }