Fix: Preserve reference image panel state and selection on recall#9010
Fix: Preserve reference image panel state and selection on recall#9010ufuksarp wants to merge 8 commits intoinvoke-ai:mainfrom
Conversation
Made-with: Cursor
|
Nice fix — the priority order (ID match → fingerprint match → first entity fallback) is the right call, and it correctly handles the config variants that don't have a 1. Unreachable fallback in the fingerprint key const imageName = config.image?.original.image.image_name ?? config.image?.crop?.image.image_name ?? '';Per the const imageName = config.image?.original.image.image_name ?? '';2. Edge case: panel open with no prior selection If 3. Fingerprint collision for empty configs Two reference images of the same |
Simplify image-name fingerprint fallback, add an explicit guard for open-panel with null selection, and document the acceptable empty-config collision tradeoff. Made-with: Cursor
|
Thanks for the feedback.
Yeah, I was aware of this. As you said, it’s very unlikely and the worst case is selecting the wrong one of the two, so I didn’t think it justified extra complexity and left a comment for now. |
Summary
Preserves reference image panel state and which reference image is selected when metadata recall replaces the reference list.
Note: This is my first PR for Invoke. I am not an experienced developer and I used Cursor mainly for this. Feedback and corrections are very welcome.
Related Issues / Discussions
Closes #8982
QA Instructions
Merge Plan
Only one file changes (
refImagesSlice.ts). Nothing about this should complicate merging on its own.Checklist
What's Newcopy (if doing a release after this PR)