-
-
Notifications
You must be signed in to change notification settings - Fork 852
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 CICP color metadata and PNG support #2592
Conversation
|
@@ -61,6 +61,7 @@ public static class Png | |||
public const string TestPattern31x31 = "Png/testpattern31x31.png"; | |||
public const string TestPattern31x31HalfTransparent = "Png/testpattern31x31-halftransparent.png"; | |||
public const string XmpColorPalette = "Png/xmp-colorpalette.png"; | |||
public const string AdamHeadsHlg = "Png/adamHeadsHLG.png"; |
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.
Looks like you need to use git rename
to ensure casing. https://github.com/SixLabors/ImageSharp/actions/runs/7032221919/job/19135513822?pr=2592#step:14:53
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.
This looks great! Thanks!
I can imagine choosing the final names for each of the enum values was an interesting experience.
Prerequisites
Description
This PR adds a
SixLabors.ImageSharp.Metadata.Profiles.Cicp.CicpProfile
class containing a CICP color profile (as per ITU H.273) to the image and frame metadata, and also PNG support for the new cICP chunk.