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

Reply via email to