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