Review Request 126207: [Compact Applet] Don't break the dialog size binding

[Kai Uwe Broulik]

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

Review request for Plasma.


Repository: plasma-desktop


Description
-------

We must not break the binding on the dialog's size, otherwise changing the preferredWidth/height of a dialog won't affect it.


Diffs
-----

  desktoppackage/contents/applet/CompactApplet.qml b5bbc70 

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


Testing
-------

I wasted hours on finding out why User Manager dialog sizing was broken, re-writing it several times /o\

Changing Layout.preferredHeight changes the dialog size, Layout.minimumHeight/Layout.maximumHeight constraints still work. Didn't notice breakage with other applets.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151130/eb8e549c/attachment-0001.html>