Review Request 119748: Properly align KickoffButtons to center

Dan Vrátil dvratil at redhat.com
Wed Aug 13 00:35:22 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119748/
-----------------------------------------------------------

(Updated Aug. 13, 2014, 2:35 a.m.)


Review request for Plasma.


Changes
-------

Now the icons and the labels are both horizontally and vertically centered in both vertical and horizontal mode \o/ (see the last screenshot)


Repository: plasma-desktop


Description
-------

The first and the last labels in Kickoff are misaligned (see the first screenshot) for some reason - even though the Label correctly fills the entire parent width, the test is still aligned to left. To workaround this issue, this patch wraps the Label into an Item - now the labels are correctly aligned in the center .

I don't have any explanation for why does this happen, nor do I understand why wrapping the stuff in Item fixes the problem, so please don't ask (but please explain it to me if you happen to know) :-).


Diffs (updated)
-----

  applets/kickoff/package/contents/ui/KickoffButton.qml 3a55c46 

Diff: https://git.reviewboard.kde.org/r/119748/diff/


Testing
-------

Tested Kickoff in vertical and horizontal mode, all tab labels are correctly centered


File Attachments (updated)
----------------

Before
  https://git.reviewboard.kde.org/media/uploaded/files/2014/08/12/cc2752ee-9684-42e4-b146-d918351951f8__kickoff-before.png
After
  https://git.reviewboard.kde.org/media/uploaded/files/2014/08/12/dd8b7151-2ab1-43de-acaf-b010d57e3efb__kickoff-after.png
After 2
  https://git.reviewboard.kde.org/media/uploaded/files/2014/08/13/3676134b-051f-4d19-93b7-d699ffa91288__kickoff-after2.png


Thanks,

Dan Vrátil

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140813/8fc553d7/attachment.html>


More information about the Plasma-devel mailing list