-
Notifications
You must be signed in to change notification settings - Fork 56
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
Any plans for merging the icc color profile support recently added to sway? #348
Comments
Happy to backport this change to swayfx early, not on my todo list, but open to PRs |
otherwise we can get it for free next sway release |
Oh I see, so swayfx always fully catches up to any changes in sway with their releases (with major changes being backported sooner)? I was wondering about how this was managed. When it comes to adding this earlier it's not that essential for me. I have gone back to sway for now but I am perfectly happy to wait for a while and I don't really have the time at the moment to work on this and I expect the same applies to others. I'm of course happy to test though :) and I am looking forward to switching when it gets there. |
Hi, sorry if I am wrong here, but considering this needs the vulkan renderer of wlr_renderer, are we sure that we can just get this for free with the next sway release? |
You could be right, I haven't dove into the implementation yet. If it does require Vulkan than yes we won't get it yet :/ |
Seems like this will be resolved by #360, rebasing off Sway 1.10. |
Yup, but if it needs the Vulkan renderer this wouldn't be the case. Still need to dive into the implementation to determine that |
Pretty much the title. I am talking about this pull request that has added the
color_profile
option for theoutput
command. I ask as it is pretty major feature for me given how oversaturated colors are without it on my screen.The text was updated successfully, but these errors were encountered: