Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

brightness control #190

Closed
acornejo opened this issue Mar 30, 2016 · 6 comments
Closed

brightness control #190

acornejo opened this issue Mar 30, 2016 · 6 comments

Comments

@acornejo
Copy link

a big factor in power consumption, especially in laptops, is screen brightness.

it would be fairly easy to add this feature to TLP, so that the user can specify default values for brightness when in AC/battery and have TLP apply them when appropriate. I would be happy to contribute a pull-request with this feature if there are changes of having it included in TLP.

Thoughts?

@FadeMind
Copy link
Contributor

In KDE Plasma this is controled via plasma-workspace and powerdevil package and settings are stored in powermanagementprofilesrc file, so this feautre is manage via current used DE.

@acornejo
Copy link
Author

@FadeMind, but in gnome shell or i3 this is not available.

@linrunner
Copy link
Owner

Hi,

such a feature would inevitably conflict with several desktop environments implementing (incomplete and buggy) their own kind of brightness control – including gnome shell (Power --> Screen brightness; Dim screen when inactive).

From a user's perspective it would always be TLP that "doesn't work" leading to many support questions or issues. As an additional complication these issues would be difficult to explain and/or solve.

Conclusion: no, thanks.

Therefore I decided long ago not to have brightness control in TLP --> see FAQ.

@acornejo
Copy link
Author

I think these concerns would be addressed by having this feature disabled by default, so users have to opt in and read a comment before enabling it; but I understand and respect your decision.

FWIW this feature has been removed from the latest gnome (I don't know why, it used to work rather well), but it is certainly available in other DE or other gnome versions.

@linrunner
Copy link
Owner

Reopened to show up on the issues page :-)

@linrunner linrunner reopened this Mar 30, 2016
@linrunner
Copy link
Owner

@acornejo: no way.

Repository owner locked and limited conversation to collaborators Mar 30, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants