WIP: bug#2783: KMilo fixing and expanding to control properly for sound level and brightness into generic monitor #8
Closed
deloptes
wants to merge 1 commits from bug/2783/kmilo
into master
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'bug/2783/kmilo'
Deleting a branch is permanent. It CANNOT be undone. Continue?
This is supplement to bug#2782
KMilo patch for fixing and expanding.
KMilo now in the generic monitor works only for sound volume control but only through DCOP Id "Mixer0" which is not everything as master card used and more, into modern configurations with PulseAudio it newer uses as the master card.
My patch:
For now I have on my notebook proper work of the shortcut keys with notifications from KMilo about the volume state, level and the brightness level.
Emanoil, thanks for taking the time to create a PR for this bug. Since the original patch is from Roman Savochenko, the commit should indicate him as author :-)
I do not mind. It was just hanging around for too long, so that I decided to push a bit.
This PR will not be merged as is. For the brightness part, I have prepared a reworked PR #21. For the mute part, I will take a look in the next few days.
Great, let me know I can test (at least the mute part).
Also this relates with the kmix patch in TDE/tdemultimedia#4 somehow - I don't remember the details now, but both had to be applied so OSD can work.
Hi Emanoil,
you can test #21 for the brightness part, I am waiting for feedback before I merge.
For the mute part I will post here when I am done with it.
This PR has been obsoleted by PR #26.