catppuccin / stable-diffusion-webui

🧑‍🎨 Soothing pastel theme for Stable Diffusion WebUI
MIT License
289 stars 59 forks source link

UX design implementation issues #6

Open TWIISTED-STUDIOS opened 1 year ago

TWIISTED-STUDIOS commented 1 year ago

I have noticed a few issues with the theme this uses, when you have this enabled the arrows on the boxes to change the values are missing, also when you are generating an image if you previously had some generations displayed, then when you go to generate more they are covered over with an invisible box and you can't see the rest of them until the new generation is finished. This was never a problem with not using this theme implementation. here is some examples showing the missing arrows from the values boxes and the invisible box as noted by the small images near the bottom.

image image

s-ilent commented 1 year ago

I also experience this issue when using this.

The missing arrows seem to be caused by this line https://github.com/catppuccin/stable-diffusion-webui/blob/7972ff32b81fe25ee6c70f211c267bf24c025c4f/style.css#L379 Removing it and setting the width to 5em fixes the issue. Having the arrows is useful because several of the sliders in the UI have a wide range, but move in discrete steps. For me, moving along those discrete steps is something I do more often than grabbing the bars themselves.

Kettukaa commented 1 year ago

Thanks for the issue! I can test out this fix this weekend when I have a chance. @s-ilent if you already have the code, then feel free to make a PR and I can merge it in. If not, I can get this done soon-ish.