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

feat(backlight): added percentexp as format option #1870

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Narice
Copy link
Contributor

@Narice Narice commented Dec 6, 2022

Humans don't perceive light linearly, so an exponential curve is more appropriate to gauge the perceived brightness of a screen.
This is fully tested and should work everywhere (Tested with Nix).
The new format would need to be documented on the wiki.
The icon markup is based on the normal percent markup for now but I can make an iconexp.
Also, I don't know the syntax for markup, so if it needs to be percent_exp and icon_exp (which is more readable), I can do that as well

Humans don't perceive light linearly and so an exponential curve is more
appropriate to gauge the perceived brightness of the screen.
changed syntax of percentexp to percent_exp for readability
@Narice
Copy link
Contributor Author

Narice commented Dec 6, 2022

I actually did what I said preemptively:
percentexp -> percent_exp
added icon_exp
Tested and icon_exp is actually quite nice to have

btw, I use brightnessctl to be able to change my brightness with a certain exponent:

brightnessctl -n --exponent=2.718 s +1%

@Alexays
Copy link
Owner

Alexays commented Jul 4, 2023

Just fixed the conflict, sorry for the delay, could you update the man and the Wiki? 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants