From ecbcf242d5e751f686fb127cd6759480e1dedab1 Mon Sep 17 00:00:00 2001
From: Brenno Lemos <brenno@syndel.is>
Date: Sun, 15 Oct 2023 17:50:41 -0300
Subject: [PATCH] feat: allow unmuting by moving the pulseaudio slider

---
 include/util/audio_backend.hpp    |  6 +++++
 src/modules/pulseaudio_slider.cpp | 37 +++++++++++++++++++++++++++++++
 src/util/audio_backend.cpp        | 20 +++++++++++++++++
 3 files changed, 63 insertions(+)

diff --git a/include/util/audio_backend.hpp b/include/util/audio_backend.hpp
index 9d043612..8d9b6f2f 100644
--- a/include/util/audio_backend.hpp
+++ b/include/util/audio_backend.hpp
@@ -83,6 +83,12 @@ class AudioBackend {
   std::string getSourceDesc() const { return source_desc_; }
   std::string getDefaultSourceName() const { return default_source_name_; }
 
+  void toggleSinkMute();
+  void toggleSinkMute(bool);
+
+  void toggleSourceMute();
+  void toggleSourceMute(bool);
+
   bool isBluetooth();
 };
 
diff --git a/src/modules/pulseaudio_slider.cpp b/src/modules/pulseaudio_slider.cpp
index edd92670..bf85584e 100644
--- a/src/modules/pulseaudio_slider.cpp
+++ b/src/modules/pulseaudio_slider.cpp
@@ -38,7 +38,44 @@ void PulseaudioSlider::update() {
 }
 
 void PulseaudioSlider::onValueChanged() {
+  bool is_mute = false;
+
+  switch (target) {
+    case PulseaudioSliderTarget::Sink:
+      if (backend->getSinkMuted()) {
+        is_mute = true;
+      }
+      break;
+
+    case PulseaudioSliderTarget::Source:
+      if (backend->getSourceMuted()) {
+        is_mute = true;
+      }
+      break;
+  }
+
   uint16_t volume = scale_.get_value();
+
+  if (is_mute) {
+    // Avoid setting sink/source to volume 0 if the user muted if via another mean.
+    if (volume == 0) {
+      return;
+    }
+
+    // If the sink/source is mute, but the user clicked the slider, unmute it!
+    else {
+      switch (target) {
+        case PulseaudioSliderTarget::Sink:
+          backend->toggleSinkMute(false);
+          break;
+
+        case PulseaudioSliderTarget::Source:
+          backend->toggleSourceMute(false);
+          break;
+      }
+    }
+  }
+
   backend->changeVolume(volume, min_, max_);
 }
 
diff --git a/src/util/audio_backend.cpp b/src/util/audio_backend.cpp
index eb2cfaff..7eef1448 100644
--- a/src/util/audio_backend.cpp
+++ b/src/util/audio_backend.cpp
@@ -241,6 +241,26 @@ void AudioBackend::changeVolume(ChangeType change_type, double step, uint16_t ma
   pa_context_set_sink_volume_by_index(context_, sink_idx_, &pa_volume, volumeModifyCb, this);
 }
 
+void AudioBackend::toggleSinkMute() {
+  muted_ = !muted_;
+  pa_context_set_sink_mute_by_index(context_, sink_idx_, muted_, nullptr, nullptr);
+}
+
+void AudioBackend::toggleSinkMute(bool mute) {
+  muted_ = mute;
+  pa_context_set_sink_mute_by_index(context_, sink_idx_, muted_, nullptr, nullptr);
+}
+
+void AudioBackend::toggleSourceMute() {
+  source_muted_ = !muted_;
+  pa_context_set_source_mute_by_index(context_, source_idx_, source_muted_, nullptr, nullptr);
+}
+
+void AudioBackend::toggleSourceMute(bool mute) {
+  source_muted_ = mute;
+  pa_context_set_source_mute_by_index(context_, source_idx_, source_muted_, nullptr, nullptr);
+}
+
 bool AudioBackend::isBluetooth() {
   return monitor_.find("a2dp_sink") != std::string::npos ||  // PulseAudio
          monitor_.find("a2dp-sink") != std::string::npos ||  // PipeWire