Hi everybody,<br><br>Looks like a nice addition.<br>I wonder why the i18nc call is necessary, but I trust you pano to have a good reason for that. <br>The only little remark I would have about the patch is with the member QLabel naming ("percentage") it doesn't follow the member naming convention it'd seem (it stands out in the patch, all other members are prefixed with "m_").<br>
Other than that it's fine by me.<br><br>Cheers,<br>--<br>Pierre<br><br><div class="gmail_quote">On Tue, Dec 7, 2010 at 19:35, Panagiotis Papadopoulos <span dir="ltr"><<a href="mailto:pano_90@gmx.net">pano_90@gmx.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Hello,<br>
<br>
Our Student at Google Code-In, Furkan, already completed this task:<br>
<a href="http://www.google-melange.com/gci/task/show/google/gci2010/kde/t129157984557" target="_blank">http://www.google-melange.com/gci/task/show/google/gci2010/kde/t129157984557</a><br>
<br>
It adds the current zoom percentage of the website to the zoombar.<br>
I attached his patch (which can also be found in the link above).<br>
<br>
To me code looks good, and (pretty important:) it also works, I’d just need the OK of one of you, so that I can approve his work :-)<br>
<br>
Greetings<br>_______________________________________________<br>
rekonq mailing list<br>
<a href="mailto:rekonq@kde.org">rekonq@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/rekonq" target="_blank">https://mail.kde.org/mailman/listinfo/rekonq</a><br>
<br></blockquote></div><br>