catppuccin / stable-diffusion-webui

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

Matching "Interrupt" and "Skip" button colors to accent colors #12

Closed 00b7ce closed 1 year ago

00b7ce commented 1 year ago

I noticed that the "Interrupt" and "Skip"buttons do not reflect the accent color of the theme. The source seems to do so explicitly, but in my opinion, these buttons should also match the accent color.

This change will result in the following display. (accent color: sapphire)

FpsEFXfaMAANggG

Kettukaa commented 1 year ago

Hey, sorry I just noticed the PR. I think something like this can be good, but I have some thoughts on how it should be implemented. Since interrupt and skip look nicer if they are separate close colors, it may be hard to have the theme decide the best two colors for each chosen accent color. I may need to define the best two colors for each option? We can look into this.

00b7ce commented 1 year ago

Sorry too, I closed the PR because I realized I needed to apply it to all styles. I think that the best way to deal with the color coding of the buttons is by shifting the hue. I am trying a 20 degree shift in my implementation and it feels good.