feat: added ability to preserve exif metadata, fixed a quality slider… - #14
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Adds an option to preserve EXIF metadata during image compression and fixes the JPEG quality slider value display, wiring the new setting through the UI → settings persistence → compression pipeline.
Changes:
- Add “Preserve EXIF data” setting (UI toggle, persisted in
config.json, and passed into compression). - Implement EXIF extraction/re-injection for supported output formats using
img-parts. - Fix quality slider to reflect the current JPEG quality value (instead of always starting at 100).
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/views/settings_view.rs | Adds the new “Preserve EXIF data” checkbox to Settings UI. |
| src/views/main_view.rs | Fixes initial quality slider value to use state.quality for JPEG. |
| src/services/image_service.rs | Extracts EXIF from source and re-injects into output via img-parts. |
| src/models/language.rs | Adds localized string key for the new setting label. |
| src/components/settings.rs | Adds preserve_exif to persisted settings with default value. |
| src/components/app.rs | Wires new toggle message + passes preserve_exif into compression task; adjusts settings window height. |
| languages/*.json | Adds preserveExifData translations for all embedded languages. |
| README.md | Documents metadata preservation feature and new dependency. |
| Cargo.toml | Adds img-parts dependency; bumps rust-version. |
| Cargo.lock | Locks img-parts and transitive dependency updates. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
… display issue