[rekonq] Re: Review Request: Fix icon duplication

[Benjamin Poulain]

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



src/urlbar/urlbar.cpp
<http://git.reviewboard.kde.org/r/100067/#comment76>

    I am not a fan of this condition.
    I would edit WebTab to have a method isPageLoading() returning a boolean. The condition could then be rewritten as 
    if (!_tab->isPageLoading()) {
    
    If not the above method, I would at least add a comment. Progress == 0 is too cryptic in my opinion.


- Benjamin


On 2010-10-17 17:55:24, Yoann Laissus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100067/
> -----------------------------------------------------------
> 
> (Updated 2010-10-17 17:55:24)
> 
> 
> Review request for rekonq.
> 
> 
> Summary
> -------
> 
> Fix icon duplication in the urlbar if a bookmark is added during a page loading
> 
> 
> Diffs
> -----
> 
>   src/urlbar/urlbar.cpp 4039e8b 
> 
> Diff: http://git.reviewboard.kde.org/r/100067/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Yoann
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/rekonq/attachments/20101018/73805d9a/attachment-0001.htm