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.

Sign up for GitHub

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

Jump to bottom

Daytime kelvin gets reset when going into settings if you are above slider maximum #308

Closed
4 of 5 tasks
JdotCarver opened this issue May 31, 2024 · 1 comment · Fixed by #312
Closed
4 of 5 tasks

Daytime kelvin gets reset when going into settings if you are above slider maximum #308

JdotCarver opened this issue May 31, 2024 · 1 comment · Fixed by #312
Labels
bug

Comments

@JdotCarver
Copy link

JdotCarver commented May 31, 2024 •
edited
Loading

Version

2.5.1

Platform

Windows 10

Steps to reproduce

image

  • Go into app settings
  • Set "Day-time color temperature:" to 8200
  • Tab
  • The change occurs
  • Click save
  • Come back to settings
  • It's back to 6600

Details

This used to work fine.
And this currently works for the "Night-time color temperature" slider.

Checklist

  • I have looked through existing issues to make sure that this bug has not been reported before
  • I have provided a descriptive title for this issue
  • I have made sure that this bug is reproducible on the latest version of the application
  • I have provided all the information needed to reproduce this bug as efficiently as possible
  • I have sponsored this project
@JdotCarver JdotCarver added the bug label May 31, 2024
@JdotCarver JdotCarver changed the title Cannot set daytime kelvin to above slider maximum anymore Daytime kelvin gets reset when going into settings if you are above slider maximum May 31, 2024
Tyrrrz added a commit that referenced this issue May 31, 2024
@Tyrrrz
Adjust slider bounds for day/night temperature dynamically, based on …
4f84168 
…the current values

Closes #308
@Tyrrrz Tyrrrz mentioned this issue May 31, 2024
Merged
@Tyrrrz
Copy link
Owner

Tyrrrz commented May 31, 2024

The fix will be available in the next release. Thanks for reporting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
2 participants
@Tyrrrz @JdotCarver