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

(Updated Sept. 3, 2012, 4:46 p.m.)


Review request for Plasma, Aurélien Gâteau and Marco Martin.


Changes
-------

Changed to ColumnProxyModel, I think it was the less bad name.

The rest was more or less agreed I think, is it ok to push this? We can still 
iterate with it in master.


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 (updated)
-----

  plasma/declarativeimports/qtextracomponents/CMakeLists.txt 05a1195 
  plasma/declarativeimports/qtextracomponents/columnproxymodel.h PRE-CREATION 
  plasma/declarativeimports/qtextracomponents/columnproxymodel.cpp PRE-CREATION 
  plasma/declarativeimports/qtextracomponents/qtextracomponentsplugin.cpp 
429282e 
  plasma/declarativeimports/qtextracomponents/tests/CMakeLists.txt PRE-CREATION 
  plasma/declarativeimports/qtextracomponents/tests/columnproxymodeltest.h 
PRE-CREATION 
  plasma/declarativeimports/qtextracomponents/tests/columnproxymodeltest.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
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to