Add an Inhibit button to the Battery applet
Review Request #102757 - Created Oct. 2, 2011 and submitted
As the title says. The feature works, but I'm wondering if we can make it look better. Any suggestions warmly appreciated :)
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.
c/p error :)
Fishy ... I suspect lots of people going "WTF" when seeing a play/stop button... Should probably be something in the same style as the logout/shutdown/suspend/... buttons
Elsewhere, we use pushbuttons, which communicate better if something's enabled or not. Changing button meaning / text is usually not a very good idea, but having a button pressed or not makes it better visible.
Review request changed
Adapted the UI as per suggestion from Aaron. It now roughly looks like: Battery: 39% (discharging) AC Adapter: Not plugged in Time remaining: 50 minutes Enable power management: [ x ] Screen brightness: [============== o===========]
Revision 2 (+18 -67)
i like the direction of this towards simplicity, predictability and "making good decisions in the software" ..