Review Request 121384: Fix margins in systray
Kai Uwe Broulik
kde at privat.broulik.de
Sun Dec 7 20:04:12 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121384/
-----------------------------------------------------------
(Updated Dez. 7, 2014, 8:04 nachm.)
Review request for Plasma.
Repository: plasma-workspace
Description
-------
Now that the sidebar is hidden when item is not, stuff looks quite broken.
Diffs
-----
applets/systemtray/package/contents/ui/ExpandedRepresentation.qml c5c35bd
Diff: https://git.reviewboard.kde.org/r/121384/diff/
Testing
-------
Looks much better, not sure whether the margin between the separtor and the Passive item is enough or like it was before, but looks okay.
One severe issue remains, though (outside this patch): The thing only checks the plasmoid's status, not whether it is forcefully hidden/shown by the tray which means: clicking a Passive plasmoid that is forced to be shown shows the sidebar and vice-versa clicking a hidden plasmoid that is Active hides the sidebar with no obvious way to get back to the list.
File Attachments
----------------
Tray overview
https://git.reviewboard.kde.org/media/uploaded/files/2014/12/07/ea866377-7ccc-47cd-b1cc-b79954cb7bdb__tray_list.png
Active plasmoid
https://git.reviewboard.kde.org/media/uploaded/files/2014/12/07/23ab78b4-5774-48e1-b639-acf4452263c7__tray_activeexpand.png
Passive plasmoid
https://git.reviewboard.kde.org/media/uploaded/files/2014/12/07/ea8b8c05-4015-4dd0-b17a-3b4e6bc0a653__tray_hiddenexpand.png
Thanks,
Kai Uwe Broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141207/f0dbf441/attachment.html>
More information about the Plasma-devel
mailing list