Review Request: Plasma QML Components: fix internal anchoring in ToolButton
Sebastian Gottfried
sebastiangottfried at web.de
Wed Dec 19 10:54:41 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107813/
-----------------------------------------------------------
Review request for KDE Runtime and Marco Martin.
Description
-------
There is a bug in the tool button component resulting in layout breakage if one clears and sets the text property of the component when not visible, see the attached screenshot.
I have tried to solve the issue without changing the existing anchoring system, but without success. The only working solution was to put the icon and the label item into Column item. That way I was able to fix the bug and even get rid of the ugly explicit non-declarative anchor assignments.
I have also removed the preferredWidth property of the label item, that one always evaluated to paintedWidth, anyway.
For consistency the button component should get the same treatment, I will do so once I get positive feedback here.
Diffs
-----
plasma/declarativeimports/plasmacomponents/qml/ToolButton.qml 594067d
Diff: http://git.reviewboard.kde.org/r/107813/diff/
Testing
-------
Work as expected. No behavioral changes apart from the bugfix.
Screenshots
-----------
Broken tool button layout
http://git.reviewboard.kde.org/r/107813/s/921/
Thanks,
Sebastian Gottfried
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121219/c2c67ea3/attachment.htm>
More information about the kde-core-devel
mailing list