Review Request 120604: Cleanup applet configuration scrollbar handling

Kai Uwe Broulik kde at privat.broulik.de
Thu Oct 16 14:38:22 UTC 2014


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

(Updated Oct. 16, 2014, 2:38 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Bugs: 336533 and 337298
    https://bugs.kde.org/show_bug.cgi?id=336533
    https://bugs.kde.org/show_bug.cgi?id=337298


Repository: plasma-desktop


Description
-------

This cleans up the applet configuration scrollbar handling which does away with the spurious binding loop on the Column and binds the StaackView height to the viewport height instead of the item height doing away with the glitching scrollbar appearance.

Also gets rid of some useless anchors.


Diffs
-----

  desktoppackage/contents/configuration/AppletConfiguration.qml 6c4b7f5 

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


Testing
-------

No more binding loop warning (as far as I could see it in the verbose console output), no more once in a while appearing scrollbars when resizing dialogs smaller.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141016/19cfee60/attachment.html>


More information about the Plasma-devel mailing list