[Kde-pim] RR: Add additional parameter to EntityTreeModel::entityData()

[Stephen Kelly]

David Jarvie wrote:
>> the entityData methods are just for convenience. As you have a special
>> use case, I recommend that you override ::data instead.
> 
> Looking at the entitytreemodel.cpp code, I think you're right - in fact, I
> need to return a value for Qt::BackgroundRole also, which currently is
> handled by data() and not passed through to entityData() to handle.
> 
> I think this points to a lack of API documentation for EntityTreeModel -
> there's nothing to describe the difference between data() and
> entityData().
> 
> Something else missing from the apidox class description is how the model
> is organised - I presume that it's in a tree structure which mirrors the
> Collection/Item tree structure, but it's not actually stated.
> 

I added some more explicit information to the dox about this. 

Can you tell me how you change the background colour of items if not by 
using the EntityDisplayAttribute? Is the background colour built into the 
alarm payload itself?

All the best,

Steve.

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/