> On Dec. 4, 2014, 9:26 a.m., Kai Uwe Broulik wrote: > > Nice! > > > > I will borrow that angleDelta trick for the Plasma calendar where I used a > > similar, yet different and worse, approach [1]. > > > > The brightness percent value was there for a reason. Powerdevil only > > allowed brightness in 10 steps from 0 to 100. Keyboards tend to have less > > steps, such 3 or 4, and then you end up with it displaying 30%, where it > > really is 33% leading to unexpected brightness changes when interacting > > with the applet. However, there has been attempts to fix this [2] but I am > > not sure whether the dataengine supports that now. The battery applet > > certainly does not honor these properties yet. (I should look into that, > > though) > > > > [1] > > http://quickgit.kde.org/?p=plasma-framework.git&a=blob&h=11013a76f24725cd0b308cd8a2c26ee741fe831b&hb=c2f140fae7c37dad38b6834be7cd8075cd9116cf&f=src%2Fdeclarativeimports%2Fcalendar%2Fqml%2FMonthView.qml#l125 > > > > [2] https://git.reviewboard.kde.org/r/119597/
If you look at the code, the percent will always comes from dataengine, are you talking about when send value to dataengine, the dataengine will return a different value after actual adjustion? BTW, for the angleDelta trick, you can reduce 120 to 12 and step from 10 -> 1 to get fine-grained if we don't want 10 as step anymore. Actually I tried to remove step: 10 in slider, and I can get extremely smooth brightness change (at least my system support it). - Xuetian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121346/#review71328 ----------------------------------------------------------- On Dec. 4, 2014, 8:56 a.m., Xuetian Weng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121346/ > ----------------------------------------------------------- > > (Updated Dec. 4, 2014, 8:56 a.m.) > > > Review request for Plasma and Kai Uwe Broulik. > > > Repository: plasma-workspace > > > Description > ------- > > 1. remove redundant "percentage" brightness > 2. make wheel on tray usable on high precision touchpad: > http://qt-project.org/doc/qt-5/qml-qtquick-wheelevent.html#angleDelta-prop > See also for a bug with similar reason > https://bugs.kde.org/show_bug.cgi?id=313579 > > > Diffs > ----- > > applets/batterymonitor/package/contents/code/logic.js a5ac7f0 > applets/batterymonitor/package/contents/ui/PopupDialog.qml 0298f7f > applets/batterymonitor/package/contents/ui/batterymonitor.qml 5e61a21 > > Diff: https://git.reviewboard.kde.org/r/121346/diff/ > > > Testing > ------- > > touchpad scrolling now works in a more reasonable adjustion speed. > > > Thanks, > > Xuetian Weng > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
