LukeSmithxyz / voidrice

My dotfiles (deployed by LARBS)
GNU General Public License v3.0
4.27k stars 1.21k forks source link

Update volume module on headphone/monitor (dis)connect #1311

Closed appeasementPolitik closed 1 year ago

appeasementPolitik commented 1 year ago

fixes #1304

Requires a new application in progs.csv, acpid. There is a problem of how to enable this daemon to start on boot from install, especially considering that on artix, this application also has its own -runit, -6, etc. packages (maybe there's a different package or way of doing this better?)

If anyone finds any better way of listening to acpi messages, please give a comment.

appeasementPolitik commented 1 year ago

Another option would be to append acpid to /etc/rc.local so that it gets run on boot, it's an unusual way of starting these types of services, but it would work for both arch and artix

LukeSmithxyz commented 1 year ago

I don't think this is worth having some extra script running at all times.

I have the remap thing because users might be confused as to what happens if the keybinding revert, but this volume status bar isn't really a big issue because any interaction with the audio system or module will immediately update the icon.

appeasementPolitik commented 1 year ago

Keeping this in mind would you consider only merging the changes in sb-volume? It mirrors what happens when you press super+f4, but now also when you left click the module in the status bar

thetubster99 commented 3 months ago

Hey, nice feature. I found a way to work around acpid using pactl. See https://github.com/thetubster99/cookedrice/commit/23c36ba8b088b8d5549dd8f86ab7683a7b02e1d8