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

Dream Texture: UI Changes #39

Merged
merged 7 commits into from
Sep 21, 2022
Merged

Conversation

blastframe
Copy link
Contributor

image

  • Changed UI from what was a tedious three-click process to generate a new Dream Texture image to a one-click process.
  • "Dream Texture" text was very wide in the Image Editor & Node Editor menus & hard to access, now in collapsible sidebar
  • Organized UI module into folder
  • Assigned more appropriate History icon
  • Smaller UI tweaks

* Changed UI from what was a three-click process to generate a new Stable Diffusion to a one-click process.
* "Dream Texture" text was very wide in the Image Editor & Node Editor menus & hard to access, now in collapsible sidebar
* Organized UI module into folder
* Assigned more appropriate History icon
* Smaller UI tweaks
Copy link
Owner

@carson-katri carson-katri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really nice, thanks for the contribution! I had a few suggestions, let me know what you think.

blastframe and others added 2 commits September 20, 2022 14:01
Co-authored-by: Carson Katri <[email protected]>
Co-authored-by: Carson Katri <[email protected]>
@blastframe
Copy link
Contributor Author

Thank you @carson-katri for the feedback and incredible add-on! I've accepted the changes you requested.

@blastframe blastframe closed this Sep 20, 2022
@blastframe blastframe reopened this Sep 20, 2022
@blastframe
Copy link
Contributor Author

Oops, I accidentally closed it. It should be ready to merge!

This fixes a version check bug. The original check was comparing a tuple of strings to a tuple of ints. This converts the strings to ints.

Before: VERSION (0, 0, 5) latest_version ('0', '0', '5')
After: VERSION (0, 0, 5) latest_version (0, 0, 5)
@carson-katri
Copy link
Owner

Thanks for the extra fixes!

@carson-katri carson-katri merged commit b6e7861 into carson-katri:main Sep 21, 2022
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.

2 participants