[rekonq] Re: Review Request: Add a "Go" action

[Andrea Diamantini]

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


Can I ask what's the need for this?
if you wanna add the "go" button in the urlbar, I fear you forgot it. And I think it can be better to implement it locally in the urlbar.
If there is another goal, sorry. I cannot see it :(

- Andrea


On 2010-11-01 13:03:33, Yoann Laissus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100125/
> -----------------------------------------------------------
> 
> (Updated 2010-11-01 13:03:33)
> 
> 
> Review request for rekonq.
> 
> 
> Summary
> -------
> 
> - A go action is added to the KXMLGUI action collection.
> - A minor change in UrlBar
> 
> 
> This addresses bug 255521.
>     /show_bug.cgi?id=255521
> 
> 
> Diffs
> -----
> 
>   src/mainview.h 29162dd 
>   src/mainview.cpp beaf4e2 
>   src/mainwindow.cpp 7b4cf8a 
>   src/urlbar/urlbar.h 4cdd9d9 
>   src/urlbar/urlbar.cpp 1f4a036 
> 
> Diff: http://git.reviewboard.kde.org/r/100125/diff
> 
> 
> Testing
> -------
> 
> It works as expected.
> 
> 
> Thanks,
> 
> Yoann
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/rekonq/attachments/20101101/b9d21b28/attachment.htm