I think it would improve usability if there was an option to enable notifications when the volume is changed. volumed for example does this, as do various other tools. This is probably not necessary for the slider, but should be displayed when the mousewheel or volume keys are used.
Not a priority atm but nice to have. Patches welcome. Any extra dependencies should be optional.
Created attachment 6181 Patch adding a notification API Hi, I've made a patch that exposes a notification API to the PA plugin. It's based on volumed, but updated to use mute icons consistently with the PA plugin (mute icon when volume == 0 rather than low volume icon). I haven't made use of the API in the existing code because I'm not sure how to implement the behaviour which I think would work best. Pointers are very welcome if you want me to finish the job :-) Expected behaviour: - When keys are used to update the volume, throw a volume notification - When the mouse wheel is used on the panel plugin icon, also show a notification (the mouse covers the icon so visual feedback is mostly missing) - When the menu is open, throw no notifications (redundant feedback) If I can find out how to detect whether the menu is open from within pulseaudio_volume_set_volume_cb1, then I can probably finish the patch. I also need access to the PulseaudioNotify struct. API usage: - When starting your app, create a PulseAudioNotify with pulseaudio_notify_init() - When leaving, use pulseaudio_notify_uninit() - When updating the volume, update current_vol and muted in your PulseaudioNotify and call pulseaudio_notify_volume_notification The API stores muted/current_vol because it can make it a bit easier to update notifications. For instance if you have a callback for muting that does not have access to the current volume, you can still update the PulseaudioNotify struct and throw a notification with the correct volume. The drawback is you need to remember to update them, even when not showing a notification. It's, also, a remnant of the volumed architecture. If you prefer to have a pulseaudio_notify_volume_notification function that takes current_vol and muted as parameters, I'm happy to change the patch to do that.
Steve, thank you for the patch. In the end I have implemented it slightly differently but most of the code is based on your work. I have also added you to the list of contributors, please let me know if you are happy with the current copyright and acknowledgements.
All good for me, thanks :-)
I just installed 0.2.3, notifications work beautifully. Thanks both of you!