----------------------------------------------------------- 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 > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
