PulseVolumeWidget: Add checkbox to indicate current sink

[laomuon]

Add a checkbox to indicate current sink in PulseVolumeWidget

[elParaguayo]

Thanks! This is a great addition. Can you just include a screenshot here so I can the effect of the change?

The docs error is unrelated to you PR. Just an issue I haven't had time to fix yet.

[laomuon]

Can you just include a screenshot here so I can the effect of the change?

Hi, it would look like this

[elParaguayo]

That's a huge improvement.

I just need to fix the other issue that's broken all the tests and then we can run CI on this. Will try to do it this weekend if I can.

[elParaguayo]

@laomuon Can you rebase this onto the current main branch and force push this. I'd expect tests to pass now and I would like to get this one merged.

[elParaguayo]

Looks like you need to do this:

diff --git a/qtile_extras/widget/pulse_volume.py b/qtile_extras/widget/pulse_volume.py
index 7f60bec..a99e00d 100644
--- a/qtile_extras/widget/pulse_volume.py
+++ b/qtile_extras/widget/pulse_volume.py
@@ -75,14 +75,14 @@ class PulseVolume(QPulseVolume, MenuMixin):
 
         default_sink_name = (await pulse.pulse.server_info()).default_sink_name
         for sink in sinks:
-                menu_items.append(
-                    pmi(
-                        text=sink.description,
-                        toggle_box=True,
-                        toggled= default_sink_name == sink.name,
-                        **_callback(sink)
-                    )
+            menu_items.append(
+                pmi(
+                    text=sink.description,
+                    toggle_box=True,
+                    toggled=default_sink_name == sink.name,
+                    **_callback(sink),
                 )
+            )