Review Request: This patch add an easy way to up/down brightness only scrolling on battery plasmoid top.

[Aaron J. Seigo]

On Monday 01 December 2008, Sebastian Kügler wrote:
> On Monday 01 December 2008 14:32:39 Gökmen GÖKSEL wrote:
> > On Monday 01 December 2008 03:14:17 Aaron Seigo wrote:
> > > hm. i actually take that "Ship it!" back .. i don't think it makes any
> > > sense to change the screen brightness when scrolling over the battery
> > > icon. coupled with the fact that there is no user feedback when this
> > > happens (other than the screen getting dimmer or brighter ;) this is
> > > just asking to annoy people.
> > >
> > > click on on the battery to get the controls; how hard is that? =)
> >
> > KPowerSave was provide that and it was a very useful shortcut I guess :)
>
> I'd be OK with it as a config option, off by default (and no UI before 4.2
> because of string freeze).

sorry, what does moving the mouse wheel over the battery icon have to do with 
the brightness of my screen?

at the very least it requires onscreen notification, but it really seems like 
an odd feature.

and the problem with features is that you can't take them back once offered.

-- 
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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20081201/3f3cc9d1/attachment.sig