Review Request 111968: Make newly opened search tabs close- and editable

[Frank Reininghaus]

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

Ship it!


Thinking about it again, the current situation is even worse in the use case that I described, so this is no reason not to push the change. Better push it sooner rather than later to get it into the next unstable release and give it some testing, and look at the remaining bug later. Thanks again for your work on this issue!

- Frank Reininghaus


On March 16, 2014, 4:28 p.m., Emmanuel Pescosta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/111968/
> -----------------------------------------------------------
> 
> (Updated March 16, 2014, 4:28 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Bugs: 311950
>     http://bugs.kde.org/show_bug.cgi?id=311950
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> When you open a new tab while the search mode is enabled, the newly opened 
> tab also starts the same search (Because new tab is opened with the current 
> view url), but the search box is in read-only mode. 
> So you cannot close the search bar nor edit the search text.
> 
> This patch fixes this by parsing the search url. The value of the "search"
> parameter is used as search text and the value of the "url" parameter is used 
> for the search path ("root" folder for the search when "Search from here" mode 
> is enabled).
> 
> In case of Baloo search urls, we use Baloo::Query::fromSearchUrl.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinviewcontainer.cpp 768fd5e 
>   dolphin/src/search/dolphinfacetswidget.h 526fe1c 
>   dolphin/src/search/dolphinfacetswidget.cpp b7315a0 
>   dolphin/src/search/dolphinsearchbox.h 6dc09f1 
>   dolphin/src/search/dolphinsearchbox.cpp c178c43 
> 
> Diff: https://git.reviewboard.kde.org/r/111968/diff/
> 
> 
> Testing
> -------
> 
> 1. Start a search (e.g. "kde") in folder $HOME/Downloads
> 2. Open a new tab
> 3. Edit the search mode should work
> 4. Close the search
> 5. The view from the newly opened tab should open $HOME/Downloads
> 
> All five points works for me.
> 
> Tested with Baloo 4.13 Beta1
> 
> 
> Thanks,
> 
> Emmanuel Pescosta
> 
>

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