D8367: Hidding place groups implementation in KFilePlacesModel

Kevin Ottens noreply at phabricator.kde.org
Mon Nov 27 07:46:38 UTC 2017


ervin added inline comments.

INLINE COMMENTS

> franckarrecot wrote in kfileplacesmodel.cpp:870
> Because it wouldn't compile without the whole KFIlePlacesModel:: prefix, so I end up going for variable renaming, seemed cleaner

Let's go for naming it "groupHidden" then. It's a weird shortening otherwise.

REPOSITORY
  R241 KIO

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

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


More information about the Kde-frameworks-devel mailing list