-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4810/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

This implements feature requested in bug 230888 . Scrolling over battery 
plasmoid changes brightness.

In (uncommon) case of multiple brightness devices it affects the first 
registered device. We may want to make it configurable in the future.


This addresses bug 230888.
    https://bugs.kde.org/show_bug.cgi?id=230888


Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/generic/applets/battery/battery.h 1157714 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/battery/battery.cpp 
1157714 

Diff: http://reviewboard.kde.org/r/4810/diff


Testing
-------

It works fine for my notebook, doesn't crash on machine without brightness 
device.

The part I am not sure about is:
Applet::wheelEvent(e);
I though we need to propagate events so tried to use this call. However using 
it causes crash. Help?


Thanks,

Rafa?

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

Reply via email to