Ok so i keep the reimplentation and i remove the setFocus only, and i call the base class.<br><br><div class="gmail_quote">2008/11/26 Aaron J. Seigo <span dir="ltr"><<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>></span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div class="Wj3C7c">On Wednesday 26 November 2008, Alexis Ménard wrote:<br>
> Hello,<br>
><br>
> Here is a patch that remove the focus stuff in the mousePressEvent of<br>
> Applet class and remove the overload. Why? Because Qt have a focus policy<br>
> that manage that for you. It break the binary compatibilty but it not too<br>
> late no? :D.<br>
> I think it will avoid some future problems regarding focus.<br>
><br>
> Review please?<br>
<br>
</div></div>you can do this without breaking BC, obviously. i'd like us to get in the<br>
habit of doing so and in this case it keeps the door open for adding something<br>
to that method if needed later on.<br>
<font color="#888888"><br>
--<br>
Aaron J. Seigo<br>
humru othro a kohnu se<br>
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43<br>
<br>
KDE core developer sponsored by Qt Software<br>
<br>
</font><br>_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
<br></blockquote></div><br>