Skip to content

Conversation

@psychedelicious
Copy link
Contributor

@psychedelicious psychedelicious commented Oct 21, 2022

The context menu is a Radix UI component. I started making my own but realized the scope of handling everything, and searched for a library to use.

One of the first and most promising results was a Radix UI component, and as I was already leaning towards Radix decided to use it. Works nicely.

@psychedelicious psychedelicious changed the title Gallery improvements Web UI: Gallery improvements Oct 21, 2022
@blessedcoolant
Copy link
Collaborator

blessedcoolant commented Oct 21, 2022

A couple of changes:

  1. The background hover color in dark mode on the right click panel is too similar to the base color. Needs to be lighter.
  2. Change the cursor to pointer on the right click menu too.
  3. Add a bit of spacing between the right click elements so they dont look like a block of text.
  4. Setting use all Parameters unsets the init image breaking the display. Use all Parameters should not apply init_image at all.

@psychedelicious
Copy link
Contributor Author

@blessedcoolant
Updated styling.

We should set the init image if one was used... However I can replicate what I think is the issue you had, if the init image no longer exists.

To address this, we need to allow the image to attempt to load. Then use the onError listener to un-set the init image when it fails, popping a toast in that situation.

Does that fix the issue you ran into?

@psychedelicious
Copy link
Contributor Author

updated changes in first post.

@blessedcoolant blessedcoolant merged commit a127eef into invoke-ai:development Oct 21, 2022
@psychedelicious psychedelicious deleted the gallery-fix branch October 25, 2022 22:55
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