<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />broulik set the repository for this revision to rPOWERDEVIL Powerdevil.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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/D2775" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Just because the sysfs helper succeeded, doesn't mean we actually have screen brightness support.<br />
Only if maximum brightness greater than 0 is reported, we'll announce its support.<br />
This fixes the brightness slider showing up in the KCM on desktop computers. The battery monitor already has the same logic and correctly did not show brightness sliders in this case.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Compiles.<br />
On desktop with 5.7 right now, don't want to mess up my setup with kded powerdevil vs standalone binary ;)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPOWERDEVIL Powerdevil</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2775" rel="noreferrer">https://phabricator.kde.org/D2775</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>daemon/backends/upower/powerdevilupowerbackend.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>