----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105042/#review14213 -----------------------------------------------------------
given the comment in there about crashes and holes in models, Marco should weigh in on this. i'll poke him about it on monday. nice catch though ... - Aaron J. Seigo On May 25, 2012, 2:40 p.m., Aurélien Gâteau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105042/ > ----------------------------------------------------------- > > (Updated May 25, 2012, 2:40 p.m.) > > > 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
