D16918: [dimdisplay] Do not change screen brightness on invalid values
This revision was automatically updated to reflect the committed changes. Closed by commit R122:6f350690723a: [dimdisplay] Do not change screen brightness on invalid values (authored by anthonyfieroni). REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16918?vs=47017=48388 REVISION DETAIL https://phabricator.kde.org/D16918 AFFECTED FILES daemon/actions/bundled/dimdisplay.cpp To: anthonyfieroni, #plasma, broulik, mart Cc: thsurrel, broulik, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16918: [dimdisplay] Do not change screen brightness on invalid values
anthonyfieroni added a comment. @broulik, @thsurrel any other ideas or to commit it? REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D16918 To: anthonyfieroni, #plasma, broulik, mart Cc: thsurrel, broulik, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16918: [dimdisplay] Do not change screen brightness on invalid values
anthonyfieroni added a comment. Isn't there. https://phabricator.kde.org/source/powerdevil/browse/master/daemon/backends/upower/xrandrbrightness.cpp$160 You can investigate on what backend it's used for you and it may set value only one output not to all. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D16918 To: anthonyfieroni, #plasma, broulik, mart Cc: thsurrel, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16918: [dimdisplay] Do not change screen brightness on invalid values
thsurrel added a comment. Neither helped... The problem may not be in plasma. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D16918 To: anthonyfieroni, #plasma, broulik, mart Cc: thsurrel, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16918: [dimdisplay] Do not change screen brightness on invalid values
anthonyfieroni added a comment. @thsurrel try this patch, if it not help disable dim display from power saving configuration, is that help? REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D16918 To: anthonyfieroni, #plasma, broulik, mart Cc: thsurrel, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16918: [dimdisplay] Do not change screen brightness on invalid values
thsurrel added a comment. This bug is bothering me too and I haven't found a fix either. I use an external screen connected to my laptop and I turn my laptop screen off by setting its brightness to 0. Then whenever I go away for a moment and the screen is turned off because energy saving kicks in, when I get back to work the laptop screen brightness gets to 100%. Particularly not nice when it's night time ... REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D16918 To: anthonyfieroni, #plasma, broulik, mart Cc: thsurrel, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16918: [dimdisplay] Do not change screen brightness on invalid values
anthonyfieroni updated this revision to Diff 47017. anthonyfieroni retitled this revision from "[brightnesscontrol] Do not change screen brightness on invalid values" to "[dimdisplay] Do not change screen brightness on invalid values". REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16918?vs=45566=47017 REVISION DETAIL https://phabricator.kde.org/D16918 AFFECTED FILES daemon/actions/bundled/dimdisplay.cpp To: anthonyfieroni, #plasma, broulik, mart Cc: thsurrel, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart