Media Controls on lock screen
ClosedPublic

Authored by broulik on Dec 15 2016, 11:09 AM.

Details

Summary

This provides media controls on the lock screen including album art, track, and title names.

Test Plan

What's to be done:

  • Allow to disable this for privacy reasons
  • Prints a couple of QML warnings when the player exits as the source goes away but since you cannot quit the player from here (and it still ends up hiding the controls then) isn't too bad

Album art, title, artist, controls


No album art, no artist, in this case the player name is shown instead

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik updated this revision to Diff 9030.Dec 15 2016, 11:09 AM
broulik retitled this revision from to WIP: Media Controls on lock screen.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added reviewers: Plasma, Plasma: Design.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma. · View Herald TranscriptDec 15 2016, 11:09 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Dec 15 2016, 12:42 PM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Dec 15 2016, 12:42 PM

I can also reproduce the crash when running kscreenlocker_greet --testing

It crashes when the KActionCollection (that holds the actions for the global shortcuts) gets destroyed on teardown. By that time KGlobalAccel cleanup already happened, it then creates a new instance apparently and then crashes somewhere. Is this a DBus (lockup) issue maybe? I can't get a meaningful backtrace, though. It doesn't happen if I have it leak the KActionCollection.

broulik retitled this revision from WIP: Media Controls on lock screen to Media Controls on lock screen.Feb 14 2017, 3:21 PM
broulik edited the test plan for this revision. (Show Details)
broulik edited edge metadata.
graesslin accepted this revision.Feb 15 2017, 4:08 PM
graesslin added a reviewer: graesslin.
graesslin added a subscriber: graesslin.

From my side it can go in even without the config option. Let's better integrate the new features early and add the config lateron

This revision was automatically updated to reflect the committed changes.

thanks well done.