Skip to content

feat: add quality configuration for saving WebP images#2094

Open
tmchow wants to merge 2 commits intoPintaProject:masterfrom
tmchow:feat/webp-quality
Open

feat: add quality configuration for saving WebP images#2094
tmchow wants to merge 2 commits intoPintaProject:masterfrom
tmchow:feat/webp-quality

Conversation

@tmchow
Copy link
Copy Markdown

@tmchow tmchow commented Apr 4, 2026

Adds quality dialog for WebP saves following the JPEG pattern. Closes #1571.

This contribution was developed with AI assistance (Claude Code).

Add a quality dialog when saving WebP images, matching the existing
JPEG quality behavior. When saving a WebP file for the first time in a
session, the user is prompted with a quality slider (default 80).

- New WebPFormat class following the JpegFormat pattern
- Register WebPFormat in ImageConverterManager
- Add WEBP_QUALITY setting for persisting the quality value
- Generalize quality dialog title from "JPEG Quality" to "Image Quality"

Closes PintaProject#1571
@JGCarroll
Copy link
Copy Markdown
Contributor

I'd just suggest that JpegCompressionDialog.cs both filename and the class itself are potentially renamed to something else, now that it's not just JPEG only.

Possibly just CompressionDialog.cs with CompressionDialog as the class, although I'm thinking perhaps since the lines below refer to the concept generally as "quality" that maybe QualityDialog would be more consistent?

The dialog is now used for both JPEG and WebP quality settings,
so the JPEG-specific name no longer fits.
@tmchow
Copy link
Copy Markdown
Author

tmchow commented Apr 5, 2026

I'd just suggest that JpegCompressionDialog.cs both filename and the class itself are potentially renamed to something else, now that it's not just JPEG only.

Possibly just CompressionDialog.cs with CompressionDialog as the class, although I'm thinking perhaps since the lines below refer to the concept generally as "quality" that maybe QualityDialog would be more consistent?

Done!

@cameronwhite
Copy link
Copy Markdown
Member

This contribution was developed with AI assistance

Is this an automated contribution or have you done some work yourself?
I'll likely wait on reviewing this until we make some decisions on our policy (see also #2086)

@tmchow
Copy link
Copy Markdown
Author

tmchow commented Apr 12, 2026

This contribution was developed with AI assistance

Is this an automated contribution or have you done some work yourself? I'll likely wait on reviewing this until we make some decisions on our policy (see also #2086)

Combo of my own thoughts assisted with AI. It's a little difficult to clean split and for me to declare some % split though ofc. If contributions assisted with AI are not welcomed, I can respect that and we can close it out.

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.

Support configuring the quality when saving webp images

3 participants