Review Request: Make it possible to use Tree models from QML

[Aleix Pol Gonzalez]

> On Aug. 30, 2012, 8:34 a.m., Marco Martin wrote:
> > while it may be useful being able to access other columns, being able to specify the root index duplicates the standard
> > http://qt-project.org/doc/qt-4.8/qml-visualdatamodel.html
> > 
> > not sure it is possible combining them tough.
> > 
> > comment on name and location, i would put it in qtextracomponents since doesn't depend on plasma and call it something like TreeModelAccess?
> >

I don't think it's comparable with VisualDataModel. There you put the delegate on the model, I don't really understand why I'd do that, though....

Moving it to qtextracomponents makes total sense. TreeModelAccess sounds weird too, but Listify is worse :D.


- Aleix


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


On Aug. 30, 2012, 8:17 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. 30, 2012, 8:17 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/core/CMakeLists.txt f488f0d 
>   plasma/declarativeimports/core/corebindingsplugin.cpp 92d0bef 
>   plasma/declarativeimports/core/listifymodel.h PRE-CREATION 
>   plasma/declarativeimports/core/listifymodel.cpp PRE-CREATION 
>   plasma/declarativeimports/core/tests/CMakeLists.txt c1df83c 
> 
> 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
> 
>

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