stable-diffusion-webui icon indicating copy to clipboard operation
stable-diffusion-webui copied to clipboard

Fix text box arrows

Open dathide opened this issue 2 years ago • 2 comments

Remove Firefox section and change the width of gradio's text inputs to 6em, which is the default for this web-ui. These changes bring back the up and down arrows that can be clicked to change various values in the ui.

dathide avatar Feb 21 '23 01:02 dathide

Thanks for the issue and the PR!

This was an issue between aesthetics and functionality. I had removed the up/down arrows because they're not directly themeable with CSS and greatly hurt the aesthetic of the theme, but I understand that they do serve a practical purpose. I think a good compromise would be adding this as an optional setting? I can work on that over the next few days.

Kettukaa avatar Feb 21 '23 04:02 Kettukaa

Thanks for the issue and the PR!

This was an issue between aesthetics and functionality. I had removed the up/down arrows because they're not directly themeable with CSS and greatly hurt the aesthetic of the theme, but I understand that they do serve a practical purpose. I think a good compromise would be adding this as an optional setting? I can work on that over the next few days.

I tried to do something with the arrows dark theme-ness and on Chrome/Edge this seems to work. You need to set opacity to 1, but also need this additional selector

image

input[type="number"]:not([step="any"])::-webkit-inner-spin-button,
input[type="number"]:not([step="any"])::-webkit-outer-spin-button {
  opacity: 1;
}

younyokel avatar May 21 '23 09:05 younyokel