----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106272/#review18325 -----------------------------------------------------------
plasma/declarativeimports/qtextracomponents/fullmodelaccess.cpp <http://git.reviewboard.kde.org/r/106272/#comment14489> I am actually wondering if hitting this condition is really weird: From QML, setting the model can only be done through the rootIndex property, so if I am using the model to filter model1 and then want to change it to filter model2, I am going to pass to setRootIndex() an index which is not from the current source model. - Aurélien Gâteau On Aug. 31, 2012, 7:53 a.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106272/ > ----------------------------------------------------------- > > (Updated Aug. 31, 2012, 7:53 a.m.) > > > Review request for Plasma, Aurélien Gâteau and Marco Martin. > > > Description > ------- > > This patch adds a component called ListifyModel (yeah, I hate the name too). > The idea behind is to expose as a QAbstractListModel any part of a > QAbstractItemModel. > > This solves the problem we have in QML given the limitation that ListView > only displays the first column of the root items. Here we can specify what > column we want and what root index we want to have. > > > Diffs > ----- > > plasma/declarativeimports/qtextracomponents/CMakeLists.txt 05a1195 > plasma/declarativeimports/qtextracomponents/fullmodelaccess.h PRE-CREATION > plasma/declarativeimports/qtextracomponents/fullmodelaccess.cpp > PRE-CREATION > plasma/declarativeimports/qtextracomponents/qtextracomponentsplugin.cpp > 429282e > plasma/declarativeimports/qtextracomponents/tests/CMakeLists.txt > PRE-CREATION > plasma/declarativeimports/qtextracomponents/tests/fullmodelaccesstest.h > PRE-CREATION > plasma/declarativeimports/qtextracomponents/tests/fullmodelaccesstest.cpp > PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/106272/diff/ > > > Testing > ------- > > There's a passing unit test, albeit limited. > I also tested it with a QML example I had with KPeople. If anybody is > interested I can provide it too. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
