[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Patch to mousePressEvent in Plasma lib.
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2008-11-26 15:51:00
Message-ID: 200811260851.00603.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


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


["signature.asc" (application/pgp-signature)]

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]