D7477: Limit folder panel to home directory if inside home

Robert Hoffmann noreply at phabricator.kde.org
Wed Aug 30 19:44:02 BST 2017


hoffmannrobert marked an inline comment as done.
hoffmannrobert added inline comments.

INLINE COMMENTS

> emmanuelp wrote in folderspanel.cpp:357
> `FoldersPanel::slotLoadingCompleted` already clears the selection and thus this special case shouldn't be required.
> 
> Your patch, without this changes, works fine locally (e.g. when going from `/` to `~`). 
> Can you please provide the exact steps to make it fail? Thanks!

1. Create a subdirectory of `~` , e.g. `~/Documents`
2. go there --> `Documents` is selected in folders panel,
3. go back to `~` --> fail, selection in folders panel stays at `Documents` and is not cleared.

4. Create another subdirectory level. e.g. `~/Documents/Test`
5. go there --> `Documents/Test` is selected in folders panel,
6. go back to `~` --> fail, selection in folders panel stays at `Documents/Test` and is not cleared.

REPOSITORY
  R318 Dolphin

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

To: hoffmannrobert, #dolphin, elvisangelaccio, emmanuelp
Cc: emmanuelp, elvisangelaccio, #konqueror, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20170830/4d07654f/attachment.htm>


More information about the kfm-devel mailing list