Review Request 124841: Fix some spacing issues in NM applet
David Edmundson
david at davidedmundson.co.uk
Thu Aug 20 17:27:54 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124841/
-----------------------------------------------------------
Review request for Plasma and Jan Grulich.
Repository: plasma-nm
Description
-------
Plotter had a top margin of gridUnit / 2
axis labels assumed a topMargin of gridUnit /2, so labels would
sometimes clip the horizontal line.
Both weren't needed anyway as as we can just increase the topMargin of
the parent item instead (the one at line 285), so I did that and ported it to the more
semantic largeSpacing at the same time.
Also removed a topMargin in details view which had no top anchor so it
wasn't doing anything.
Diffs
-----
applet/contents/ui/ConnectionItem.qml 047cc9f3e4aefdd9457038ed312cdbae7e8f8a8f
Diff: https://git.reviewboard.kde.org/r/124841/diff/
Testing
-------
looks basically the same...
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150820/9676f1f5/attachment.html>
More information about the Plasma-devel
mailing list