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

Switch to the upstream color-management-v1 protocol #9443

Open
helixoid opened this issue Feb 19, 2025 · 6 comments
Open

Switch to the upstream color-management-v1 protocol #9443

helixoid opened this issue Feb 19, 2025 · 6 comments
Labels
enhancement New feature or request

Comments

@helixoid
Copy link

Description

Maybe it's time to switch to the staging color-management-v1 protocol for HDR and color management as mpv,sdl and mesa are adopting that protocol.

@helixoid helixoid added the enhancement New feature or request label Feb 19, 2025
@nnyyxxxx
Copy link
Contributor

doesnt hyprland use its own protocols

@nnyyxxxx
Copy link
Contributor

actually i think its a mix of the 2

@helixoid
Copy link
Author

doesnt hyprland use its own protocols

I think it's KDE's protocol modified to fit Hyprland
And KDE's was just the upstream protocol but in early stage without some bits which made KDE to switch to upstream a breeze!

@helixoid
Copy link
Author

But I may be wrong as I am not a contributor.
It's just a wild guess!

@UjinT34
Copy link
Contributor

UjinT34 commented Feb 19, 2025

I am on it but it won't be useful without #9064
The proto itself doesn't mean much. All CM features are optional and a compositor without any CM can claim proto support.
Current implementation only works with a limited subset of features for fullscreen apps. There is no way to reveal those feature for fullscreen apps and if a windowed one tries to use them than it'll end up with wrong colors. It's not an issue with xx-cm and frog-cm since they are experimental and have limited client support which is usually locked behind some flag. Upstream CM will have more issues with the current approach.

@nnyyxxxx
Copy link
Contributor

makes sense

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants