-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add feature to adjust fade duration (with Elektra) #1
base: improve-config/elektrify
Are you sure you want to change the base?
Changes from 3 commits
4a9fe11
ed4b787
edecb91
ab7dfd1
005ddb0
082b872
12917a2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -322,6 +322,8 @@ options_load_from_elektra( | |
|
||
// Fade | ||
options->use_fade = !elektraGetFadeFast(elektra); | ||
|
||
options->fade_duration = elektraGetFadeDuration(elektra); | ||
|
||
// Temperature | ||
options->scheme.day.temperature = elektraGetTempDay(elektra); | ||
|
@@ -398,6 +400,7 @@ options_init(options_t *options) | |
options->provider = NULL; | ||
|
||
options->use_fade = -1; | ||
options->fade_duration = -1; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. indent There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks, fixed. I also adjusted CLion IDE to better detect existing indents in files. |
||
options->preserve_gamma = 1; | ||
options->mode = PROGRAM_MODE_CONTINUAL; | ||
options->verbose = 0; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -34,6 +34,8 @@ typedef struct { | |
int temp_set; | ||
/* Whether to fade between large skips in color temperature. */ | ||
int use_fade; | ||
/* The length of the fade duration in seconds */ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should probably link to the spec "fade/duration" and not repeat the description? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, thanks for the hint! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed. |
||
long fade_duration; | ||
/* Whether to preserve gamma ramps if supported by gamma method. */ | ||
int preserve_gamma; | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is easier to understand, thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.