Ok so i keep the reimplentation and i remove the setFocus only, and i call the base class.
2008/11/26 Aaron J. Seigo <[EMAIL PROTECTED]> > On Wednesday 26 November 2008, Alexis Ménard wrote: > > Hello, > > > > Here is a patch that remove the focus stuff in the mousePressEvent of > > Applet class and remove the overload. Why? Because Qt have a focus policy > > that manage that for you. It break the binary compatibilty but it not too > > late no? :D. > > I think it will avoid some future problems regarding focus. > > > > Review please? > > you can do this without breaking BC, obviously. i'd like us to get in the > habit of doing so and in this case it keeps the door open for adding > something > to that method if needed later on. > > -- > Aaron J. Seigo > humru othro a kohnu se > GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 > > KDE core developer sponsored by Qt Software > > > _______________________________________________ > Plasma-devel mailing list > Plasma-devel@kde.org > https://mail.kde.org/mailman/listinfo/plasma-devel > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel