[rekonq] Review Request: Let the User to Load Different URL than the first item of the completetion widget

[Furkan Üzümcü]

> On Jan. 10, 2012, 9:42 a.m., Andrea Diamantini wrote:
> > Sorry Furkan, but this change basically cuts out any logic from orderlist imposing a definite order. Pushed wrong patch?

What about the new patch?
This way we don't show the qulrFromUserInput user loads whaever he/she wants which is the default behaviour Firefox. And I like it very much. :)


- Furkan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103652/#review9697
-----------------------------------------------------------


On Jan. 10, 2012, 10:04 a.m., Furkan Üzümcü wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103652/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2012, 10:04 a.m.)
> 
> 
> Review request for rekonq.
> 
> 
> Description
> -------
> 
> Currently, when you type something in the URL bar the completetion widget pops up and activates the first item. And you can't load the URL typed in when you hit enter. Rekonq loads the actvated item. This patch fixes that issue. When you reached the index zero if you go up again the last item gets deactivated and you can load the typed string.
> 
> 
> Diffs
> -----
> 
>   src/urlbar/completionwidget.h 517a7ea 
>   src/urlbar/completionwidget.cpp b349041 
>   src/urlbar/urlresolver.cpp 5bf2ee6 
> 
> Diff: http://git.reviewboard.kde.org/r/103652/diff/diff
> 
> 
> Testing
> -------
> 
> Compiles and works
> 
> 
> Thanks,
> 
> Furkan Üzümcü
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/rekonq/attachments/20120110/ef538b31/attachment.html>