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


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?


- Marco Martin


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
> 
>

_______________________________________________
Plasma-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to