[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 Nate Graham changed: What|Removed |Added Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas |ma/powerdevil/commit/6fd617 |ma/powerdevil/commit/053f92 |9b63d17548ae2dc144afa96d86c |ccd69d851b843bf8d043875fc9b |da5957e |9821379 --- Comment #9 from Nate Graham --- Git commit 053f92ccd69d851b843bf8d043875fc9b9821379 by Nate Graham, on behalf of Kai Uwe Broulik. Committed on 09/10/2020 at 02:52. Pushed by ngraham into branch 'Plasma/5.20'. Honor animation brightness threshold also for sysfs helper We should only animate the screen brightness if there are enough steps available to make for a smooth transition. This check was forgotten for the new sysfs brightness animation. FIXED-IN: 5.20.0 (cherry picked from commit 6fd6179b63d17548ae2dc144afa96d86cda5957e) M +6-0daemon/backends/upower/backlighthelper.cpp M +3-1daemon/backends/upower/powerdevilupowerbackend.cpp https://invent.kde.org/plasma/powerdevil/commit/053f92ccd69d851b843bf8d043875fc9b9821379 -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 Kai Uwe Broulik changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED|RESOLVED Latest Commit||https://invent.kde.org/plas ||ma/powerdevil/commit/6fd617 ||9b63d17548ae2dc144afa96d86c ||da5957e Version Fixed In||5.20.0 --- Comment #8 from Kai Uwe Broulik --- Git commit 6fd6179b63d17548ae2dc144afa96d86cda5957e by Kai Uwe Broulik. Committed on 08/10/2020 at 19:55. Pushed by ngraham into branch 'master'. Honor animation brightness threshold also for sysfs helper We should only animate the screen brightness if there are enough steps available to make for a smooth transition. This check was forgotten for the new sysfs brightness animation. FIXED-IN: 5.20.0 M +6-0daemon/backends/upower/backlighthelper.cpp M +3-1daemon/backends/upower/powerdevilupowerbackend.cpp https://invent.kde.org/plasma/powerdevil/commit/6fd6179b63d17548ae2dc144afa96d86cda5957e -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 Bug Janitor Service changed: What|Removed |Added Status|REOPENED|ASSIGNED --- Comment #7 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/plasma/powerdevil/-/merge_requests/26 -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 Kai Uwe Broulik changed: What|Removed |Added Status|NEEDSINFO |REOPENED Resolution|WAITINGFORINFO |--- Ever confirmed|0 |1 --- Comment #6 from Kai Uwe Broulik --- I was pretty sure it would not animate for less than a certain amount of steps. -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 --- Comment #5 from Bug Janitor Service --- Dear Bug Submitter, This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information as soon as possible and set the bug status as REPORTED. Due to regular bug tracker maintenance, if the bug is still in NEEDSINFO status with no change in 30 days the bug will be closed as RESOLVED > WORKSFORME due to lack of needed information. For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging If you have already provided the requested information, please mark the bug as REPORTED so that the KDE team knows that the bug is ready to be confirmed. Thank you for helping us make KDE software even better for everyone! -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 --- Comment #4 from petrk --- @Kai Uwe Broulik 15 -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 David Edmundson changed: What|Removed |Added Resolution|--- |WAITINGFORINFO CC||k...@davidedmundson.co.uk Status|REPORTED|NEEDSINFO --- Comment #3 from David Edmundson --- Marking as needs info -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 Nate Graham changed: What|Removed |Added CC||n...@kde.org --- Comment #2 from Nate Graham --- Can you add a screen recording showing what it looks like for you? Either way the correct solution would be to fix the flickering, not revert the feature entirely. -- You are receiving this mail because: You are watching all bug changes.
[Powerdevil] [Bug 426883] Please revert smooth brightness transition patch. Epilepsy risk.
https://bugs.kde.org/show_bug.cgi?id=426883 Kai Uwe Broulik changed: What|Removed |Added CC||k...@privat.broulik.de --- Comment #1 from Kai Uwe Broulik --- How many brightness steps does your laptop support? Please provide the output of $ qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/BrightnessControl org.kde.Solid.PowerManagement.Actions.BrightnessControl.brightnessMax -- You are receiving this mail because: You are watching all bug changes.
