provide an info popup when opening sounds folder

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "lull"
 description = "a looping sound player for generating atmospheric soundscapes"
-version = "1.1.0"
+version = "1.0.4"
 authors = ["Max Bradbury <max@tinybird.info>"]
 repository = "https://tinybird.dev/max/lull"
 license = "MIT"
@@ -9,12 +9,9 @@ edition = "2018"
 
 [dependencies]
 dirs = "^3.0.1"
-eframe = "^0.11.0"
-egui = "^0.11.0"
-gdk = "^0.13.2"
-gio = "^0"
-gtk = "^0"
-orbtk = "^0.2.31"
+gdk = "^0.14.3"
+gio = "^0.14.8"
+gtk = "^0.14.3"
 rodio = "^0.11.0"
 serde = "^1.0.125"
 serde_derive = "^1.0.125"

deploy.sh

@@ -1,5 +1,8 @@
 #!/usr/bin/env bash
 
+rm -rf dist/windows
+mv package dist/windows
+
 butler push dist/linux ruin/lull:linux
 butler push dist/windows ruin/lull:windows
 butler push dist/sounds ruin/lull:sounds

src/main.rs

@@ -1,17 +1,9 @@
 #![windows_subsystem = "windows"]
 
 mod config;
-mod ui;
+mod ui_gtk;
 
 use std::path::PathBuf;
-use std::io::BufReader;
-use std::fs::File;
-
-use rodio::{Sink, Source};
-use std::process::Command;
-
-const SOUNDS_BUTTON_LABEL: &'static str = "open sounds folder";
-const SPACING: i32 = 16;
 
 #[cfg(target_os = "windows")]
 fn file_manager() -> &'static str {
@@ -29,11 +21,6 @@ fn get_sounds_dir() -> PathBuf {
     data_dir.push("ruin");
     data_dir.push("lull");
 
-    // on windows, data dir and config dir are both User\AppData\Roaming
-    #[cfg(target_os = "windows")] {
-        data_dir.push("sounds");
-    }
-
     if !data_dir.exists() {
         std::fs::create_dir_all(&data_dir)
             .expect("Couldn't create lull sounds directory");
@@ -42,71 +29,7 @@ fn get_sounds_dir() -> PathBuf {
     data_dir
 }
 
-fn open_file_manager() {
-    let mut file_manager = Command::new(file_manager());
-    file_manager.arg(crate::get_sounds_dir());
-    file_manager.output().unwrap();
-}
-
-struct Sound {
-    name: String,
-    sink: Sink,
-    volume: f64,
-}
-
-fn get_sounds(device: &rodio::Device) -> Vec<Sound> {
-    let mut sounds = Vec::new();
-
-    let mut paths = std::fs::read_dir(get_sounds_dir())
-        .expect("Couldn't read lull sounds directory")
-        .map(|res| res.map(|e| e.path()))
-        .collect::<Result<Vec<_>, std::io::Error>>()
-        .expect("Couldn't read files from lull sounds directory");
-
-    paths.sort();
-
-    for path in paths {
-        let name = path.file_stem().unwrap().to_str().unwrap().into();
-
-        let file = File::open(&path)
-            .expect("Couldn't open audio file");
-
-        let source = rodio::Decoder::new(
-            BufReader::new(file)
-        );
-
-        #[cfg(not(target_os = "windows"))]
-        if source.is_err() {
-            ui::gtk::error_popup(&format!(
-                "Couldn't parse file {}. \n{}.",
-                path.to_str().unwrap(),
-                source.err().unwrap()
-            ));
-            continue;
-        }
-
-        let source = source.unwrap().repeat_infinite();
-
-        let sink = rodio::Sink::new(&device);
-        sink.append(source);
-        sink.pause();
-
-        sounds.push(Sound { name, sink, volume: 0.0 });
-    }
-
-    sounds
-}
-
-// #[cfg(not(target_os = "windows"))]
+//#[cfg(not(target_os = "windows"))]
 fn main() {
-    crate::ui::gtk::instantiate();
+    ui_gtk::instantiate();
 }
-
-// #[cfg(target_os = "windows")]
-// fn main() {
-//     eframe::run_native(Box::new(lull::ui::egui::State::default()));
-// }
-
-// fn main() {
-//     crate::ui::orbtk::instantiate();
-// }

src/ui/egui.rs

@@ -1,79 +0,0 @@
-use eframe::{egui, epi};
-
-pub(crate) use crate::{Sound, get_sounds, open_file_manager};
-use egui::{Color32, Stroke, Vec2};
-
-const COLOUR_BG: Color32 = Color32::from_rgb(76, 58, 78);
-const COLOUR_FG: Color32 = Color32::from_rgb(156, 143, 109);
-
-pub struct State {
-    device: rodio::Device,
-    sounds: Vec<Sound>,
-}
-
-impl Default for State {
-    fn default() -> Self {
-        let device = rodio::default_output_device().unwrap();
-        let sounds = get_sounds(&device);
-
-        Self { device, sounds }
-    }
-}
-
-impl epi::App for State {
-    fn update(&mut self, ctx: &egui::CtxRef, _frame: &mut epi::Frame<'_>) {
-        let State { device: _device, sounds} = self;
-
-        for sound in sounds.into_iter() {
-            if sound.volume > 0.0 {
-                sound.sink.set_volume(sound.volume as f32);
-                sound.sink.play();
-            } else {
-                sound.sink.pause();
-            }
-        }
-
-        egui::CentralPanel::default().frame(egui::Frame {
-            margin: Vec2 { x: crate::SPACING as f32, y: crate::SPACING as f32 },
-            corner_radius: 0.0,
-            shadow: Default::default(),
-            fill: COLOUR_BG,
-            stroke: Stroke {
-                width: 1.0,
-                color: COLOUR_FG,
-            }
-        }).show(ctx, |ui| {
-            egui::Grid::new("grid")
-                .spacing([crate::SPACING as f32; 2])
-                .show(ui, |ui| {
-                    for sound in sounds {
-                        ui.add(
-                            egui::Label::new(&sound.name).text_color(COLOUR_FG)
-                        );
-
-                        ui.add(
-                            egui::Slider::new(
-                                &mut sound.volume,
-                                0.0..=1.0
-                            ).show_value(false)
-                        );
-
-                        ui.end_row();
-                    }
-
-                    if ui.add(
-                        egui::Button::new(crate::SOUNDS_BUTTON_LABEL)
-                            .text_color(COLOUR_FG)
-                    ).clicked() {
-                        open_file_manager();
-                    }
-
-                    ui.end_row();
-                });
-        });
-    }
-
-    fn name(&self) -> &str {
-        "lull"
-    }
-}

src/ui/mod.rs

@@ -1,3 +0,0 @@
-// pub mod egui;
-pub mod gtk;
-// pub mod orbtk;

src/ui/orbtk.rs

@@ -1,15 +0,0 @@
-use orbtk::{prelude::*, widgets::themes::*};
-
-pub(crate) fn instantiate() {
-    Application::new()
-        .window(|ctx| {
-            Window::new()
-                .title("OrbTk - showcase example")
-                .position((100, 100))
-                .size(1000, 730)
-                .resizeable(true)
-                .child(MainView::new().build(ctx))
-                .build(ctx)
-        })
-        .run();
-}

src/ui_gtk.rs

@@ -1,16 +1,18 @@
-use std::env::args;
+use std::fs::File;
+use std::io::{BufReader};
+use std::process::Command;
 
 use gio::prelude::*;
 use gtk::prelude::*;
 use gtk::Orientation;
+use rodio::{Sink, Source};
 
-use crate::open_file_manager;
-use crate::config::load_config;
+const SPACING: i32 = 16;
 
-pub(crate) 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_border_width(crate::SPACING as u32);
+    popup.set_title(title);
+    popup.set_border_width(SPACING as u32);
     popup.set_position(gtk::WindowPosition::Center);
     popup.set_default_size(256, 64);
     popup.set_type_hint(gdk::WindowTypeHint::Dialog);
@@ -19,39 +21,49 @@ pub(crate) 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) {
     let window = gtk::ApplicationWindow::new(application);
 
     window.set_title("lull");
-    window.set_border_width(crate::SPACING as u32);
+    window.set_border_width(SPACING as u32);
     window.set_position(gtk::WindowPosition::Center);
     window.set_default_size(256, 128);
 
-    if let Some(config) = load_config() {
+    if let Some(config) = crate::config::load_config() {
         window.move_(config.position.0, config.position.1);
         window.resize(config.size.0, config.size.1);
     }
 
-    let button_manage_sounds = gtk::Button::with_label(crate::SOUNDS_BUTTON_LABEL);
+    let button_manage_sounds = gtk::Button::with_label("open sounds folder");
 
     button_manage_sounds.connect_clicked(|_| {
-        open_file_manager();
+        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();
     });
 
-    let sounds_manage  = gtk::Box::new(Orientation::Vertical,   crate::SPACING);
-    let columns        = gtk::Box::new(Orientation::Horizontal, crate::SPACING);
-    let column_labels  = gtk::Box::new(Orientation::Vertical,   crate::SPACING);
-    let column_sliders = gtk::Box::new(Orientation::Vertical,   crate::SPACING);
+    let sounds_manage  = gtk::Box::new(Orientation::Vertical,   SPACING);
+    let columns        = gtk::Box::new(Orientation::Horizontal, SPACING);
+    let column_labels  = gtk::Box::new(Orientation::Vertical,   SPACING);
+    let column_sliders = gtk::Box::new(Orientation::Vertical,   SPACING);
 
     columns.set_homogeneous(false);
     column_labels.set_homogeneous(true);
     column_sliders.set_homogeneous(true);
-    column_labels.set_property_expand(false);
-    column_sliders.set_property_expand(true);
-    column_sliders.set_property_width_request(128);
+    column_labels.set_expand(false);
+    column_sliders.set_expand(true);
+    column_sliders.set_width_request(128);
 
     window.add(&sounds_manage);
     sounds_manage.add(&columns);
@@ -61,10 +73,48 @@ fn build_ui(application: &gtk::Application) {
 
     let device = rodio::default_output_device().unwrap();
 
-    let sounds = crate::get_sounds(&device);
+    let mut paths = std::fs::read_dir(crate::get_sounds_dir())
+        .expect("Couldn't read lull sounds directory")
+        .map(|res| res.map(|e| e.path()))
+        .collect::<Result<Vec<_>, std::io::Error>>()
+        .expect("Couldn't read files from lull sounds directory");
 
-    for sound in sounds {
-        let label = gtk::Label::new(Some(sound.name.as_ref()));
+    paths.sort();
+
+    for path in paths {
+        let name      = path.file_stem().unwrap().to_str().unwrap();
+        let extension = path.extension().unwrap().to_str().unwrap();
+
+        // on Windows the data dir and config dir are the same,
+        // so avoid parsing the config file as an audio file
+        if extension == "toml" {
+            continue;
+        }
+
+        let file = File::open(&path)
+            .expect("Couldn't open audio file");
+
+        let source = rodio::Decoder::new(
+            BufReader::new(file)
+        );
+
+        if source.is_err() {
+            popup("error", &format!(
+                "Couldn't parse file {}. \n{}.",
+                path.to_str().unwrap(),
+                source.err().unwrap()
+            )).show_all();
+
+            continue;
+        }
+
+        let source = source.unwrap().repeat_infinite();
+
+        let sink = Sink::new(&device);
+        sink.append(source);
+        sink.pause();
+
+        let label = gtk::Label::new(Some(name));
         label.set_halign(gtk::Align::End);
         column_labels.add(&label);
 
@@ -85,13 +135,13 @@ fn build_ui(application: &gtk::Application) {
         slider.set_draw_value(false);
 
         slider.connect_value_changed(move |scale| {
-            let volume = scale.get_value();
+            let volume = scale.value();
 
             if volume == 0. {
-                sound.sink.pause();
+                sink.pause();
             } else {
-                sound.sink.play();
-                sound.sink.set_volume(volume as f32);
+                sink.play();
+                sink.set_volume(volume as f32);
             }
         });
 
@@ -102,8 +152,8 @@ fn build_ui(application: &gtk::Application) {
 
     window.connect_delete_event(|window, _event| {
         crate::config::save_config(crate::config::Config {
-            position: window.get_position(),
-            size:     window.get_size()
+            position: window.position(),
+            size:     window.size()
         });
 
         Inhibit(false)
@@ -120,11 +170,11 @@ pub(crate) fn instantiate() {
     let application = gtk::Application::new(
         Some("dev.tinybird.max.lull"),
         Default::default()
-    ).expect("Initialisation failed...");
+    );
 
     application.connect_activate(|app| {
         build_ui(app);
     });
 
-    application.run(&args().collect::<Vec<_>>());
+    application.run();
 }