Review Request 115500: Set minimumWidth of button to actually fit the text

[David Edmundson]

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

(Updated Feb. 5, 2014, 4 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Repository: plasma-framework


Description
-------

Set minimumWidth of button to actually show the text

The minimumWidth of a plasma button is based on the paintedWidth
of the text item. There is no way to work out the correct width it
should be if we set the text to elide.
(except for the hack of using a hidden fake text item)

We should always fit contents. This won't cause any problems unless a
developer has a broken layout, at which point that should be fixed


Diffs
-----

  src/declarativeimports/plasmacomponents/qml/Button.qml 03fa42c 

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


Testing
-------


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140205/10d7e7c4/attachment.html>