[Differential] [Commented On] D3957: Reimplement support for property groups, add API to show/hide groups

[Jarosław Staniek]

staniek added a comment.


  @piggz It would be good to have this BIC change on Monday.

INLINE COMMENTS

> piggz wrote in KPropertyEditorDataModel.cpp:399
> Full reset needed?

I think so, the model radically changes: d->indicesForNames is refreshed.

What we can add here *later* is selecting again the same row and ensuring it's visible. This needs a bit larger testing.

> piggz wrote in KPropertySet.h:235
> Add inline?

Changed to non-inline

REPOSITORY
  R13 KProperty

REVISION DETAIL
  https://phabricator.kde.org/D3957

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: staniek, piggz
Cc: Kexi-Devel-list, staniek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kexi-devel/attachments/20170116/426684f7/attachment.html>