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

[Emmanuel Pescosta]

> On Aug. 10, 2013, 10:44 a.m., Frank Reininghaus wrote:
> > Thanks for the patch!
> > 
> > Ah, it looks like setSearchModeEnabled() gets confused when the URL navigator's URL is a search URL already. Good catch! Your solution looks good to me. 
> > 
> > I'm also having trouble to get a Nepomuk search to run with my kde-devel user at the moment ("Could not find virtuoso to connect to. Aborting" is the message that appears in Konsole). But I guess that's because the virtuoso that's running here on my system is outdated, I'll try to check if upgrading helps.
> > 
> > In any case, I cannot imagine that your patch can break anything about Nepomuk searches, so it should be safe for the KDE/4.11 branch (I guess it's open again, 4.11.0 tagging was a few days ago). Thanks for your work!
> 
> Emmanuel Pescosta wrote:
>     Hmm I tested it with nepomuk search urls and it doesn't work.
>     
>     I have tried to write a nepomuk search url "extractor" to get the search text, search path, ... 
>     
>     But I haven't found a way to turn back a Nepomuk2::Query::Term which is parsed with the Nepomuk2::Query::QueryParser::parseQuery parser into the original search text.

> But I haven't found a way to turn back a Nepomuk2::Query::Term which is parsed with the Nepomuk2::Query::QueryParser::parseQuery parser into the original search text.

I think this problem is really easy to solve with Baloo (What I have seen so far) ;)

I'll update this patch once Baloo is in Dolphin master.


- Emmanuel


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


On Aug. 9, 2013, 7:48 p.m., Emmanuel Pescosta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/111968/
> -----------------------------------------------------------
> 
> (Updated Aug. 9, 2013, 7:48 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).
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinviewcontainer.cpp 1e9e79a 
> 
> 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.
> 
> Btw:
> Please test it with nepomuk searches, because nepomuk doesn't work for me (Don't know why?! - Maybe smth. is wrong in the arch/kde-unstable repo)
> 
> 
> Thanks,
> 
> Emmanuel Pescosta
> 
>

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