D5583: [Lock Screen] Show Previous/Next buttons only if supported
Kai Uwe Broulik
noreply at phabricator.kde.org
Tue Apr 25 15:49:54 UTC 2017
broulik created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
If the user can neither go back nor forward, hide those controls for a cleaner look.
TEST PLAN
If one action (e.g. only "Next" is supported), both buttons are shown with the Back button grayed out. But for the case of a simple stream or so where really only play/pause is supported, it makes it much cleaner:
F3734636: Screenshot_20170425_174729.png <https://phabricator.kde.org/F3734636>
(I initially did this for Task manager but here I found it looked odd with the tiny lost play icon F3734638: Screenshot_20170425_174546.png <https://phabricator.kde.org/F3734638>
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D5583
AFFECTED FILES
lookandfeel/contents/lockscreen/MediaControls.qml
To: broulik, #plasma, #vdg
Cc: plasma-devel, spstarr, progwolff, Zren, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170425/4e8be89e/attachment-0001.html>
More information about the Plasma-devel
mailing list