D9610: Revive folderpanel when outside $HOME

Michael Heidelbach noreply at phabricator.kde.org
Tue Jan 2 19:56:39 GMT 2018


michaelh added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in kfileitemmodel.cpp:651
> It's not possible to ends with '/' at that point but i'm searching for solution without setting empty paths
> 
>   urlToExpand.setPath(path.left(pos - 1));
> 
> will set path to empty string.

What about enforcing a trailing slash in panels/folders/folderspanel.cpp
and use

  urlToExpand.setPath(urlToExpand.path()  + subDirs.at(i) + '/'

?

  void KFileItemModel::expandParentDirectories(const QUrl &url)

is used only once anyway
	​

REPOSITORY
  R318 Dolphin

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

To: michaelh, #dolphin
Cc: elvisangelaccio, ngraham, anthonyfieroni, michaelh, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180102/acad80dc/attachment.htm>


More information about the kfm-devel mailing list