docs: record user feedback fixes for ImageAdjustmentModal
This commit is contained in:
@@ -51,11 +51,28 @@ Integrated the pre-built ImageAdjustmentModal component into the AIOnboarding it
|
||||
- [ ] E2E test: Full flow with test images
|
||||
- [ ] Mobile test: Touch gestures on adjustment modal
|
||||
|
||||
### User Feedback & Fixes Applied (Same Session)
|
||||
|
||||
**Issue 1: Modal canvas showing cropped image**
|
||||
- Root cause: Fixed zoom level (1.0) didn't account for large image dimensions
|
||||
- Fix: Calculate initial zoom to fit full image in 800×600 canvas
|
||||
- Result: Full item now visible from start, users understand what they're adjusting
|
||||
|
||||
**Issue 2: Adjustments not being applied to final image**
|
||||
- Root cause: State updates not persisted before calling confirmation hook
|
||||
- Fix: Updated handleImageAdjustmentConfirm to explicitly update extractedItems state
|
||||
- Changes stored: rotation_degrees, crop_bounds (with user_adjusted flag)
|
||||
- Result: Adjustments now properly override AI-detected values
|
||||
|
||||
**Commits:**
|
||||
- `e6a64bb4` - Fix adjustment overrides (state persistence)
|
||||
- `97629dec` - Fix canvas display (auto-fit zoom)
|
||||
|
||||
### Next Steps
|
||||
1. Test modal interaction with test images
|
||||
2. Verify touch/pinch zoom works on mobile
|
||||
3. Add backend support for applying adjustments
|
||||
4. E2E test complete flow: capture → adjust → save
|
||||
1. Test modal with real images to verify fixes work
|
||||
2. Verify touch/pinch zoom works on mobile
|
||||
3. Monitor backend for user_adjusted flag (apply overrides correctly)
|
||||
4. E2E test: capture → adjust → save → verify image saved with adjustments
|
||||
|
||||
---
|
||||
|
||||
|
||||
Reference in New Issue
Block a user