Review Request 123253: dolphin: Navigate to parent folder selects child folder

[Emmanuel Pescosta]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123253/#review103732
-----------------------------------------------------------


Ship it!




Sorry, missed your changes after the previous `ship it`

- Emmanuel Pescosta


On July 27, 2017, 1:46 p.m., Gregor Mi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123253/
> -----------------------------------------------------------
> 
> (Updated July 27, 2017, 1:46 p.m.)
> 
> 
> Review request for Dolphin and Emmanuel Pescosta.
> 
> 
> Bugs: 335616
>     https://bugs.kde.org/show_bug.cgi?id=335616
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> This is a first working implementation of the feature suggestion filed in the ticket https://bugs.kde.org/show_bug.cgi?id=335616: "Dolphin: Navigate to parent folder selects child folder".
> 
> In short, this is what is does: Whenever the dolphin view is initialized to show the contents of a new URL (e.g. "/home/x/test") it will be checked if the new URL is a parent of the previous displayed URL (e.g. "/home/x/test/documents/aaa"). If the check is successful, then the common child (in this example: "/home/x/test/documents/") folder item will be selected and scrolled into view.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 1ab84b95053c1e198ac67133366343e4bc629d2a 
>   src/dolphinviewcontainer.h e50386ab6388174b31ded5d662ab25fbeff5a990 
>   src/dolphinviewcontainer.cpp 72ced931b709f21d95848b541372b618df8edab5 
> 
> 
> Diff: https://git.reviewboard.kde.org/r/123253/diff/6/
> 
> 
> Testing
> -------
> 
> - unit test passes
> - Played around with dolphin: enter URL manually, navigate via click in the item view, navigate via click in kurlnavigator, navigate with Alt+Left, Alt+Right, Alt+up, Backspace
> 
> 2016-02-18:
> There is now a difference in selection behaviour when using history buttons vs. direct url entry:
> 
> * history buttons: mark the item as current instead of restoring the previous selection
> * direct url entry: the child item is marked as current and selected
> 
> I personally prefer the selection because it is clearly visible. If it is ok I would try to adapt the history behaviour accordingly.
>   Update: I just saw that the "File Open" dialog does not do any selection at all if navigated back via history buttons. I'll postpone further action on this side issue for now.
> 
> 
> Thanks,
> 
> Gregor Mi
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20170921/6497f139/attachment.htm>