Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix sending to img2img #76

Closed
wants to merge 20 commits into from
Closed

Fix sending to img2img #76

wants to merge 20 commits into from

Conversation

AlUlkesh
Copy link

The current pull request #51 seems to fix the problem only for some people some of the time.

This version fixes it for everyone who has tried it. #69

@WozniakMac
Copy link

WozniakMac commented Jan 24, 2023

Can we merge this?

@NoCrypt
Copy link

NoCrypt commented Jan 25, 2023

Oh sh*t, I've debugged this for 6 hours straight, found the exact same solution. Should have checked pull request before lmao

@AlUlkesh
Copy link
Author

Well, glad to hear you came to the same conclusion.

AlUlkesh and others added 11 commits January 26, 2023 07:42
Others tab:
- Add/Remove from saved directories, via buttons
- New dropdown with subdirs
- path_recorder.txt is sorted on add/remove
- path_recorder.txt is first worked on in a .tmp version, before replacing the original file, to prevent a potential loss of all data in the file

Settings:
- Usage of path_recorder.txt no longer optional
- Option to not show the images from subdirs

Install:
- Fixed send2trash module check
@temirov
Copy link

temirov commented Feb 1, 2023

Is this repo abandoned? @yfszzx

@temirov
Copy link

temirov commented Feb 1, 2023

I have removed the existing stable-diffusion-webui-images-browser folder, and used @AlUlkesh repo to manually install the fixed version

image

@catboxanon
Copy link

catboxanon commented Feb 2, 2023

Might want to re-open this PR on another branch because it changes a lot more than advertised now.

@AlUlkesh
Copy link
Author

AlUlkesh commented Feb 2, 2023

@catboxanon Yes, you're right. I hadn't thought about someone only wanting this fix now. New PR is here: #84

@AlUlkesh AlUlkesh closed this Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants