feat: add support to change the volume scrolling on the volume icon

Author: tglman

src/clients/volume/mod.rs

@@ -28,6 +28,8 @@ pub enum Event {
     AddInput(SinkInput),
     UpdateInput(SinkInput),
     RemoveInput(u32),
+    VolumeUp(f64),
+    VolumeDown(f64),
 }
 
 #[derive(Debug)]
@@ -308,4 +310,18 @@ pub fn percent_to_volume(target_percent: f64) -> u32 {
     }
 }
 
+pub fn scroll_to_volume(current_value: u32, scroll_value: f64) -> u32 {
+    let mut val: i32 = current_value as i32;
+    let scroll_delta = scroll_value * -1000.0;
+    trace!("base+delta: {val} + {scroll_delta} orig: scroll_value");
+    val += scroll_delta as i32;
+    if val > Volume::NORMAL.0 as i32 {
+        Volume::NORMAL.0 as u32
+    } else if val < Volume::MUTED.0 as i32 {
+        Volume::MUTED.0 as u32
+    } else {
+        val as u32
+    }
+}
+
 register_client!(Client, volume);

src/clients/volume/sink.rs

@@ -1,4 +1,7 @@
-use super::{ArcMutVec, Client, ConnectionState, Event, percent_to_volume, volume_to_percent};
+use super::{
+    ArcMutVec, Client, ConnectionState, Event, percent_to_volume, scroll_to_volume,
+    volume_to_percent,
+};
 use crate::{lock, send};
 use libpulse_binding::callbacks::ListResult;
 use libpulse_binding::context::Context;
@@ -53,6 +56,40 @@ impl Client {
         }
     }
 
+    #[instrument(level = "trace")]
+    pub fn set_default_volume(&self, value: f64) {
+        trace!("raceived volume change: {value:?}");
+        let mut active = None;
+        for sync in &*lock!(self.data.sinks) {
+            if sync.active {
+                active = Some(sync.name.to_string());
+                break;
+            }
+        }
+        if let Some(name) = active {
+            if let ConnectionState::Connected { introspector, .. } = &mut *lock!(self.connection) {
+                let (tx, rx) = mpsc::channel();
+
+                introspector.get_sink_info_by_name(&name, move |info| {
+                    let ListResult::Item(info) = info else {
+                        return;
+                    };
+                    send!(tx, info.volume);
+                });
+
+                let mut volume = rx.recv().expect("to receive info");
+                for v in volume.get_mut() {
+                    let dval = v.0;
+                    let val = scroll_to_volume(v.0, value);
+                    trace!("changing value from: {dval:?} to: {val}");
+                    v.0 = val;
+                }
+
+                introspector.set_sink_volume_by_name(&name, &volume, None);
+            }
+        }
+    }
+
     #[instrument(level = "trace")]
     pub fn set_sink_volume(&self, name: &str, volume_percent: f64) {
         if let ConnectionState::Connected { introspector, .. } = &mut *lock!(self.connection) {

src/modules/volume.rs

@@ -6,9 +6,12 @@ use crate::modules::{
 };
 use crate::{glib_recv, lock, module_impl, send_async, spawn, try_send};
 use glib::Propagation;
+use gtk::gdk::{EventMask, ScrollDirection};
 use gtk::pango::EllipsizeMode;
 use gtk::prelude::*;
-use gtk::{Button, CellRendererText, ComboBoxText, Label, Orientation, Scale, ToggleButton};
+use gtk::{
+    Button, CellRendererText, ComboBoxText, EventBox, Label, Orientation, Scale, ToggleButton,
+};
 use serde::Deserialize;
 use std::collections::HashMap;
 use tokio::sync::mpsc;
@@ -128,9 +131,11 @@ pub enum Update {
 
     InputVolume(u32, f64),
     InputMute(u32, bool),
+    InputVolumeUp(f64),
+    InputVolumeDown(f64),
 }
 
-impl Module<Button> for VolumeModule {
+impl Module<EventBox> for VolumeModule {
     type SendMessage = Event;
     type ReceiveMessage = Update;
 
@@ -143,7 +148,7 @@ impl Module<Button> for VolumeModule {
         mut rx: mpsc::Receiver<Self::ReceiveMessage>,
     ) -> color_eyre::Result<()>
     where
-        <Self as Module<Button>>::SendMessage: Clone,
+        <Self as Module<EventBox>>::SendMessage: Clone,
     {
         let client = context.client::<volume::Client>();
 
@@ -197,6 +202,8 @@ impl Module<Button> for VolumeModule {
                     Update::SinkVolume(name, volume) => client.set_sink_volume(&name, volume),
                     Update::SinkMute(name, muted) => client.set_sink_muted(&name, muted),
                     Update::InputVolume(index, volume) => client.set_input_volume(index, volume),
+                    Update::InputVolumeUp(val) => client.set_default_volume(val),
+                    Update::InputVolumeDown(val) => client.set_default_volume(val),
                     Update::InputMute(index, muted) => client.set_input_muted(index, muted),
                 }
             }
@@ -209,17 +216,17 @@ impl Module<Button> for VolumeModule {
         self,
         context: WidgetContext<Self::SendMessage, Self::ReceiveMessage>,
         info: &ModuleInfo,
-    ) -> color_eyre::Result<ModuleParts<Button>>
+    ) -> color_eyre::Result<ModuleParts<EventBox>>
     where
-        <Self as Module<Button>>::SendMessage: Clone,
+        <Self as Module<EventBox>>::SendMessage: Clone,
     {
         let button_label = Label::builder()
             .use_markup(true)
             .angle(self.layout.angle(info))
             .justify(self.layout.justify.into())
             .build();
 
-        let button = Button::new();
+        let button = Button::builder().build();
         button.add(&button_label);
 
         {
@@ -251,6 +258,49 @@ impl Module<Button> for VolumeModule {
             });
         }
 
+        let event_box = EventBox::builder()
+            .events(
+                EventMask::SCROLL_MASK
+                    | EventMask::SMOOTH_SCROLL_MASK
+                    | EventMask::BUTTON_MOTION_MASK,
+            )
+            .child(&button)
+            .build();
+
+        {
+            let tx = context.tx.clone();
+            event_box.connect_scroll_event(move |_button, scroll| {
+                match scroll.direction() {
+                    ScrollDirection::Up => {
+                        try_send!(
+                            tx,
+                            ModuleUpdateEvent::Update(Event::VolumeUp(scroll.delta().1))
+                        );
+                    }
+                    ScrollDirection::Down => {
+                        try_send!(
+                            tx,
+                            ModuleUpdateEvent::Update(Event::VolumeDown(scroll.delta().1))
+                        );
+                    }
+                    ScrollDirection::Smooth => {
+                        if scroll.scroll_deltas().unwrap_or_default().1 > 0.0 {
+                            try_send!(
+                                tx,
+                                ModuleUpdateEvent::Update(Event::VolumeUp(scroll.delta().1))
+                            );
+                        } else {
+                            try_send!(
+                                tx,
+                                ModuleUpdateEvent::Update(Event::VolumeDown(scroll.delta().1))
+                            );
+                        }
+                    }
+                    _ => {}
+                }
+                Propagation::Stop
+            });
+        }
         let popup = self
             .into_popup(
                 context.controller_tx.clone(),
@@ -260,7 +310,7 @@ impl Module<Button> for VolumeModule {
             )
             .into_popup_parts(vec![&button]);
 
-        Ok(ModuleParts::new(button, popup))
+        Ok(ModuleParts::new(event_box, popup))
     }
 
     fn into_popup(
@@ -460,6 +510,13 @@ impl Module<Button> for VolumeModule {
                             input_container.remove(&ui.container);
                         }
                     }
+                    Event::VolumeUp(val) => {
+                        try_send!(tx, Update::InputVolumeUp(val));
+                    }
+
+                    Event::VolumeDown(val) => {
+                        try_send!(tx, Update::InputVolumeDown(val));
+                    }
                 }
             });
         }