Repository navigation
Wait for Optimize; switching settings no longer re-runs anything - #21
Merged
Merged
Conversation
Dropped images are now read and listed (thumbnail, size, role) but not made into anything until Optimize is pressed. Pressing it while images are still being read includes them once they're read. - CMS master is the default and comes first. The output isn't remembered between visits; the other settings still are. - Changing the output, role, quality or maximum width runs nothing. Images that need new files show "Ready", and Optimize counts them. - Each image keeps its last finished set per mode. Switching back to settings it was made with shows those files again without a second run. Sets are matched on the exact variants planned, so a change that wouldn't alter the files (e.g. a role with the same widths) keeps them. - While optimizing, the settings and row roles are locked, and Stop ends the run. Finished images keep their files; the rest go back to ready. - One button carries the next step, Optimize and then Download ZIP. It uses aria-disabled while busy, so focus stays on it. Copy and Save to folder appear once everything is made with the current settings. The e2e tests press Optimize. A new test covers waiting, locking, Stop and resuming, and the mode test checks that switching back needs no second run. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01STHRjFYWhkJBxSKG3tJdap
Deploying image-optimizer with
|
| Latest commit: |
35325ce
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://cddf77a3.image-optimizer-7ba.pages.dev |
| Branch Preview URL: | https://feat-optimize-button.image-optimizer-7ba.pages.dev |
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.
Covers three of your four points. (The dropdown colours are in #20.)
Nothing runs until Optimize
Switching CMS master ↔ Responsive set no longer runs anything
One file by default
Screens (light, desktop): Ready → Optimizing (settings faded, Stop) → Done (Save to folder, Download ZIP). The phone-width dark layout wraps cleanly. I checked these by screenshot; nothing else moved.
Tests
Your call
Preview: https://feat-optimize-button.image-optimizer-7ba.pages.dev (once CI has deployed it).
🤖 Generated with Claude Code
https://claude.ai/code/session_01STHRjFYWhkJBxSKG3tJdap