kjslag added a comment.

  @romangg The current code selects from a discrete set of brightness levels 
which starts at zero. I was suggesting just removing the zero brightness level 
and using the next level as the minimum brightness level.
  
  I didn't realize turning off the screen at the lowest brightness level was an 
intended feature. My past 3 laptops didn't do that, and I've never seen a 
laptop do that on any operating system. It seems like an unintuitive feature 
and could confuse someone who doesn't realize that their screen is black 
because the brightness is all the way down. Perhaps it would be better to add a 
different hotkey for this function. (That's what I did.)
  
  Also note that setting brightness=0 probably isn't a power efficient way to 
turn the screen off. My laptop uses 4W less energy when I turn the screen off 
with
  xset dpms force off
  than when I set brightness=0.

REPOSITORY
  R122 Powerdevil

REVISION DETAIL
  https://phabricator.kde.org/D13095

To: thsurrel, #plasma, broulik, ngraham
Cc: kjslag, sharvey, zzag, ngraham, romangg, plasma-devel, jraleigh, GB_2, 
ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to