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

[Renato Oliveira Filho]

renatoo accepted this revision.
renatoo added a comment.


  Looks good and work as expected

INLINE COMMENTS

> kfileplacesview.cpp:766
> +    }
>      if (!clickOverHeader && index.isValid()) {
>          if (!placesModel->isDevice(index)) {

I think that you can replace

if (!clickOverHeader && index.isValid()) {

with

else if (index.isValid()) { 
....
}

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/c976db15/attachment.html>