Skip to content

stealth png info (again)#871

Merged
mcmonkey4eva merged 3 commits intomcmonkeyprojects:masterfrom
Hugs288:master
Jul 6, 2025
Merged

stealth png info (again)#871
mcmonkey4eva merged 3 commits intomcmonkeyprojects:masterfrom
Hugs288:master

Conversation

@Hugs288
Copy link
Contributor

@Hugs288 Hugs288 commented Jul 6, 2025

adds stealth png info support

this implementation only works with png, it is compatible with webp in theory but either sixlabors doesnt support that or stupid me just cant get it to work.

especially tricky for lossy webp since it needs the alpha layer to be encoded losslessly, which sixlabors doesnt seem to support at all.

also small change in imagemetadatatracker because webp metadata seems fixed now


/// <summary>File format extensions that even can have metadata on them.</summary>
public static HashSet<string> ExtensionsWithMetadata = ["png", "jpg"];
public static HashSet<string> ExtensionsWithMetadata = ["png", "jpg", "webp"];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unrelated change shouldn't be here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"also small change in imagemetadatatracker because webp metadata seems fixed now"

@mcmonkey4eva mcmonkey4eva merged commit f7b7a30 into mcmonkeyprojects:master Jul 6, 2025
FreesoSaiFared pushed a commit to FreesoSaiFared/SwarmUI that referenced this pull request Oct 22, 2025
* stealth png info (again)

* minor cleanups

* missed cleanups

---------

Co-authored-by: Alex "mcmonkey" Goodwin <git_commits@alexgoodwin.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants