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

Review request for Plasma.


Description
-------

SortFilterModel::setRegExp() calls beginResetModel() before setting the regexp 
and endResetModel() when done. This cause flickering as the QML view must 
refresh its entire content. QSortFilterProxyModel correctly add or remove rows 
when filtering, so resetting the model is not necessary.


Diffs
-----

  plasma/declarativeimports/core/datamodel.cpp 97d129e 

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


Testing
-------

Test script attached demonstrates the problem by having a delegate whose 
opacity goes from 0 to 1 when completed.

- Download regexp-flicker.{pro,cpp,qml}
- run qmake ; make
- Type in the input field => without the patch, remaining items flicker. With 
the patch they do not because they are not recreated.


Thanks,

Aurélien Gâteau

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

Reply via email to