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

Review request for Plasma and Marco Martin.


Description
-------

This patch changes two things:
- When non-interactive or not enabled, don't make the handle responsible of the 
flickable position.
- When the Flickable.contentHeight property changes, update the value according 
to Flickable.contentY. This is important because if both properties are updated 
at the same time we get in weird states because of RangeModel.

Also, I'd like this patch to get in for 4.9 (4.9.1 if too late for 4.9.0 :P). 
It is important because without this KTP's QuickChat Plasmoid is __very__ 
annoying.


Diffs
-----

  plasma/declarativeimports/plasmacomponents/qml/ScrollBar.qml 430de05 

Diff: http://git.reviewboard.kde.org/r/105808/diff/


Testing
-------

Played around with http://proli.net/meu/kde/testcase.qml
Also made sure it fixes my issue in ktp-text-ui


Thanks,

Aleix Pol Gonzalez

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to