Just because the sysfs helper succeeded, doesn't mean we actually have screen brightness support.
Only if maximum brightness greater than 0 is reported, we'll announce its support.
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.
Details
Details
- Reviewers
bshah - Group Reviewers
Plasma - Commits
- R122:03717dc7388b: Emit brightnessSupportQueried with the proper value
On my laptop still works. if I fake xrandr brightness not being supported, goes through sysfs helper. If I fake sysfs helper getting a 0 max brightness, controls stop working and also hide from the KCM
Diff Detail
Diff Detail
- Repository
- R122 Powerdevil
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.