The color correction manager doesn't make any specific assumptions about
underlying platform, e.g. whether it's x11, etc. The platform just
has to be capable of setting gamma ramps. Given that, there are no any
significant technical blockers for making this feature work on x.
Details
- Reviewers
davidedmundson romangg - Group Reviewers
KWin - Commits
- R108:0d381846f1cc: Backport Night Color feature to X11
Diff Detail
- Repository
- R108 KWin
- Branch
- native-redshift
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 12073 Build 12091: arc lint + arc unit
This was the original plan when redshift was started.
May as well get the extra testing on the code that is there.
If you need this patch in master right now, feel free to land it for me. I got sick with the flu so it might take some time before I do this.
Though I advise you to wait until p-w and p-d patches are accepted.
Oh no, feel better soon! We can wait a bit, no rush. I'm just eager to announce it. :)
Which other patches should we wait for? I don't see them mentioned in the Description section or listed as dependencies.
Next time just add the diffs from p-w / p-d as dependent ones or better create an overview task linking the relevant diffs, so there are none to be missed by reviewers.
I don't like the GammaRamp class being in the AbstractOutput file, but it's not that crucial.
Depends on how you look at it. Gamma ramp is a property of an output, so I see nothing wrong with keeping both together. Am I missing something?