D8450: User can now hide an entire places group from KFilePlacesView

[Laurent Montel]

mlaurent accepted this revision.
mlaurent added inline comments.

INLINE COMMENTS

> ervin wrote in kfileplacesview.cpp:863-866
> This duplicates code from the next if branch below... I wonder if before or after that commit we shouldn't try to refactor that and remove some of the code duplication around the delegate animation use?

Ok for removing duplicate code after this patch. Too hard to add patch before it.

REPOSITORY
  R241 KIO

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

To: franckarrecot, ngraham, renatoo, ervin, mwolff, mlaurent
Cc: mwolff, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171206/e3e0be39/attachment.html>