[PATCH] pass mouse wheel events to the wallpaper

Benoit Jacob jacob.benoit.1 at gmail.com
Thu Sep 4 20:13:30 CEST 2008


OK, committed in 857101.

Benoit

2008/9/4 Aaron J. Seigo <aseigo at kde.org>:
> On Thursday 04 September 2008, Benoit Jacob wrote:
>> Hi,
>>
>> The attached patch makes Containment forward mouse wheel events first
>> to the wallpaper, if the event screenPos() indicates that the mouse
>> cursor was on the wallpaper. I adapted this code from what you were
>> already doing with mouseMoveEvent().
>
> cool ... two points:
>
> you need to re-accept the the event if the wallpaper ignores it; so this:
>
> if(event->isAccepted()) return;
>
> should be:
>
>    if (event->isAccepted()) {
>        return;
>    }
>
>    event->accept();
> }
>
> the second point is that the whitespace usage around the if statement is
> incorrectin your patch (and fixed in the example above)
>
> otherwise .. beauty =) please commit.
>
> --
> 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 Trolltech
>
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
>


More information about the Plasma-devel mailing list