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

Doesn't poll for current brightness upon invocation. #2

Open
RokeJulianLockhart opened this issue Jul 12, 2023 · 6 comments
Open

Doesn't poll for current brightness upon invocation. #2

RokeJulianLockhart opened this issue Jul 12, 2023 · 6 comments

Comments

@RokeJulianLockhart
Copy link

If I change my display brightness using https://github.com/davidhi7/ddcci-plasmoid

image

and then switch to https://github.com/JonMagon/plasma-screendimmer

image

the wrong brightness is displayed.

@davidhi7
Copy link

I just randomly stumbled across this. My widget modifies the monitor brightness using DDC/CI, so basically on the hardware side, whereas this widget simply dims the picture on the software side. So since the widgets do different things, one won't affect the other.

@JonMagon
Copy link
Owner

Yes, the widgets are quite different. I don't see the point of using both together. I made this one mostly for myself, as my monitor doesn't support DDC/CI

@RokeJulianLockhart
Copy link
Author

RokeJulianLockhart commented Jul 14, 2023

Actually, @davidhi7, yours only dims the software brightness for my monitor, an AOC Q3279WG5B. When I perform systemctl poweroff, the brightness returns to the value it was before https://github.com/davidhi7/ddcci-plasmoid set it during the shutdown sequence. https://github.com/JonMagon/plasma-screendimmer, to my knowledge.

@davidhi7
Copy link

This is really strange, I've never heard anything like this before. What monitor are we talking about? Also, feel free to open a bug report in my repository to address this.

@RokeJulianLockhart
Copy link
Author

RokeJulianLockhart commented Jul 14, 2023

I don't see the point of using both together.

@JonMagon, I use yours on my laptop because https://github.com/davidhi7/ddcci-plasmoid doesn't work by default (I expect due to davidhi7/ddcci-plasmoid#21 (comment)) whereas https://github.com/JonMagon/plasma-screendimmer does. Additionally, on my desktop, I can slide your widget's slider and the changes immediately update. The other, this is not true for.

@RokeJulianLockhart
Copy link
Author

RokeJulianLockhart commented Jul 14, 2023

@davidhi7, #2 (comment)

What monitor are we talking about?

@RokeJulianLockhart, #2 (comment)

an AOC Q3279WG5B


Also, feel free to open a bug report in my repository to address this.

Thanks. Done so - davidhi7/ddcci-plasmoid#36 (comment).

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

No branches or pull requests

3 participants