plasma volume: global shortcuts and osd

[Harald Sitter]

On Wed, Jul 22, 2015 at 7:16 PM, Martin Graesslin <mgraesslin at kde.org> wrote:
> On Wednesday, July 22, 2015 06:55:42 PM Aleix Pol wrote:
>> On Wed, Jul 22, 2015 at 5:13 PM, Harald Sitter <sitter at kde.org> wrote:
>> > On Wed, Jul 22, 2015 at 4:48 PM, Aleix Pol <aleixpol at kde.org> wrote:
>> >> On Wed, Jul 22, 2015 at 11:10 AM, Harald Sitter <sitter at kde.org> wrote:
>> >>> random update before akademy.
>> >>>
>> >>> d_ed asked me to perhaps land the volume thingy for 5.4 so to make
>> >>> that happen the applet UI is going to stay minimal and the KCM UI is
>> >>> going to stay a clone of the pavucontrol UI (which isn't the greatest
>> >>> of things but functionally pretty complete).
>> >>> both UIs should be more or less complete in master of
>> >>> scratch/sitter/plasma-volume-control. I encourage testing.
>> >>>
>> >>> also I added global shortcuts and OSD support which should be the two
>> >>> major features that were missing. also kindly note that no measures
>> >>> are being taken to replace kmix right now so with kmix running you'll
>> >>> get two OSDs, other than that everything should be fine.
>> >>
>> >> Hi Harald,
>> >> I've been using it for a while, and it works reasonably well.
>> >>
>> >> It's good to see that the shortcuts started working. For me, it works
>> >> for the increase and decrease volume, but it doesn't for the mute
>> >> mic/speakers buttons. Is that known?
>> >
>> > Looked into it, the code itself is working (there was a name mapping
>> > problem but I doubt that would cause it?). At least on my laptop
>> > however the mute key doesn't actually appear to work, setting a
>> > working key combo OTOH fixes it, so in theory it should work :/
>> >
>> > Have you checked that the right code ends up in xev at all? Perhaps
>> > also try with the fix I just pushed. Maybe it was a mapping problem
>> > after all.
>> >
>> > HS
>> > _______________________________________________
>> > Plasma-devel mailing list
>> > Plasma-devel at kde.org
>> > https://mail.kde.org/mailman/listinfo/plasma-devel
>>
>> I'm not sure what you want me to check. :D
>>
>> This is what xev says [1], and the shortcuts dialog seems to work with
>> the shortcuts dialog.
>
> That sounds correct to me: it's the key code I expected and I know is properly
> mapped to Qt::Key_VolumeMute

Yeah, makes literally no sense. I blame kglobalaccel :P
Does it work when you change the shortcut to something other than
volumemute? i.e. open global hotkey kcm and change the mute hotkey in
the Audio Volume component.

HS