D9242: Implemented support for hide/show groups

[Elvis Angelaccio]

elvisangelaccio added a comment.


  Patch seems to work fine, nice job!

INLINE COMMENTS

> placespanel.cpp:340
> +    KFilePlacesModel::GroupType groupType = m_model->groupType(index);
> +    QAction *hideGroupAction = menu->addAction(i18nc("@item:inmenu", "Hide Section '%1'", m_model->item(index)->group()));
> +    hideGroupAction->setCheckable(true);

Should we add this new action/option in dolphin_placespanelsettings.kcfg ?

REPOSITORY
  R318 Dolphin

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

To: renatoo, franckarrecot, mlaurent, mwolff
Cc: rkflx, mwolff, elvisangelaccio, ngraham, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171212/9685ec5a/attachment.htm>