> On Oct. 3, 2011, 10:18 a.m., Sebastian Kügler wrote:
> > A few remarks inline. I'm not really happy with it UI wise yet, and I'd like to see a screenshot of it in action. Especially that row of iconitems can already grow quite wide, imagine hibernate and suspend both available, and the whole thing in Russian: you need a wide screen display for it ;)
> >
> > So needs some work.
>
> Dario Freddi wrote:
> Agreed, that's why I submitted this review :P unfortunately I wrongly pushed this commit to master, but well --' Any suggestions on where to place it, considering also the combo box is going to go away?
"inhibit" and "uninhibit" are things we understand as developers; i'm not sure many run-of-the-mill computer users will. it would probably be better to find some descriptive language for this that describes what is actually does, which is essentially "suspend power management features".
it should also not go alng with hibernate and suspend since those put the entire machine into an "off" state. the control for inhibiting pm should be closer to what it really controls, currently the drop down menu of choices.
it might make sense to simply note: "Power management features are on." with a button next to it change that. or a checkbox solution something like:
Battery: 39% (discharging)
AC Adapter: Not plugged in
Current strategy: Power saving
Power Management: [ x ] Enabled
Screen brightness: [============== o===========]
when the "Enabled" checkbox is unchecked, the current strategy label would grey out, reflecting it's disabled state. would require some user testing, but would seem more obvious that what is currently there to me.
btw: what will the combo box be replaced with? or why is it determined to not be of use anymore? is simple inhibition going to replace switching between profiles, under the assumption that one always wants either power management as a function of power available or else they want a full performance profile with no power management happening? (that's something i think might make sense; though i haven't done much fact gathering to support the idea ...)
- Aaron J.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102757/#review7016
-----------------------------------------------------------
On Oct. 2, 2011, 2:48 p.m., Dario Freddi wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102757/
> -----------------------------------------------------------
>
> (Updated Oct. 2, 2011, 2:48 p.m.)
>
>
> Review request for Plasma and Sebastian Kügler.
>
>
> Description
> -------
>
> As the title says. The feature works, but I'm wondering if we can make it look better. Any suggestions warmly appreciated :)
>
>
> Diffs
> -----
>
> plasma/generic/applets/battery/battery.h d696014
> plasma/generic/applets/battery/battery.cpp c3a4f58
>
> Diff: http://git.reviewboard.kde.org/r/102757/diff/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Dario Freddi
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111003/3143af4c/attachment-0001.html>