<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a subscriber: thsurrel.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19678">View Revision</a></tr></table><br /><div><div><p>-1 for making this user-configurable, sorry. The brightness curve is an implementation detail and I don't think exposing it to the user makes sense. With a string like "Use quadratic curve for brightness keys:", that's just gonna confuse users. Most Regular Peopleā„¢ don't even know what the word "quadratic" means.</p>

<p>I would support improving the curve without making it user-configurable. <em>However:</em> the last time someone proposed this in <a href="https://phabricator.kde.org/D13095" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D13095: Scren brightness follow a quatratic progression</a> it ran into a problem where for certain screens (including mine), two keypresses were required to get the backlight to turn on again after reducing the brightness to the lowest level. If you can solve that, then I'll be happy. :) Maybe you and <a href="https://phabricator.kde.org/p/thsurrel/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@thsurrel</a> can put your heads together and figure out a solution!</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19678">https://phabricator.kde.org/D19678</a></div></div><br /><div><strong>To: </strong>vishaw, Plasma, ngraham<br /><strong>Cc: </strong>thsurrel, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>