Also handle muted property on player
ClosedPublic

Authored by broulik on Apr 30 2019, 7:16 PM.

Details

Summary

Report a Volume of 0.0 for a muted player and set its muted property to whether volume is 0.0

Test Plan

Unfortunately there's no Muted property on MPris, so you cannot easily "restore" the previous volume, but that's an issue that affects every player, so when you click mute on a player and then adjust the volume in media controller it raises the volume from 0 rather than just working on the previous volume like the volume plasmoid would

  • Clicked mute on a video player, mpris reported zero, and UI reflected muted state by crossing out the volume icon on the player
  • Used wheel on media controller to change volume, media player became muted when volume hit 0.0

Shouldn't have any compatibility side-effects since we assume muted to be false by default so if this field isn't present because mixing older host with newer extension, nothing would change in behavior here.

Diff Detail

Repository
R856 Plasma Browser Integration
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Apr 30 2019, 7:16 PM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 30 2019, 7:16 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Apr 30 2019, 7:16 PM
broulik edited the test plan for this revision. (Show Details)

What do you think about adding a x-kde-muted prop and support for it in KDE Connect and the media control applet?

fvogt accepted this revision.Apr 30 2019, 7:55 PM

What do you think about adding a x-kde-muted prop and support for it in KDE Connect and the media control applet?

Sounds like a good idea.

This revision is now accepted and ready to land.Apr 30 2019, 7:55 PM
This revision was automatically updated to reflect the committed changes.