AlUlkesh / stable-diffusion-webui-images-browser

an images browse for stable-diffusion-webui
622 stars 111 forks source link

Delete can delete wrong image #159

Closed AlUlkesh closed 1 year ago

AlUlkesh commented 1 year ago

@JoeyLearnsToCode:

But now I find another problem, the "Generation Info" part update with 1s delay after switching image(which is realtime and fast). Within that second of delay, delete button seems to delete the previous image(before switch), not the new one(after).

I already fixed this bug twice. I wonder what changed to make it reappear :anguished:

JoeyLearnsToCode commented 1 year ago

I blame it on gradio instead of image browser.

AlUlkesh commented 1 year ago

Yeah, I'm actually impressed the original author was able to pull this off, despite the massive Gradio limitations.

This commit should fix this.

bryanray commented 1 year ago

please make the delete option using "recycle bin" as default :)

AlUlkesh commented 1 year ago

I have no idea, why this wasn't the default option already... I changed it, thanks.

bryanray commented 1 year ago

thanks a lot!