AUTOMATIC1111 / stable-diffusion-webui

Stable Diffusion web UI
GNU Affero General Public License v3.0
142.36k stars 26.85k forks source link

Stable Diffusion checkpoint dropdown list in quick settings on top is broken #8955

Closed f-rank closed 1 year ago

f-rank commented 1 year ago

Is there an existing issue for this?

What happened?

Clicking the Stable Diffusion checkpoint is now broken. Moment it is clicked it shows the list from the first checkpoint present in the model directory and doesn't show the list in a way that it displays the currently used checkpoint on the shown part of the listing.

So you have to scroll down and find it, each and every single time. Also, the spacing is now borked. So it became much less readable, mainly because of the vertical space introduced between lines. Maybe it has something to do with horizontal space limits it now has, I dunno, it just became unusable.

image

Steps to reproduce the problem

  1. Have the Stable Diffusion checkpoint dropdown on the top
  2. CLick it
  3. Argh intensifies

What should have happened?

Should have shown model, including the currently selected one, so user doesn't have to go fish for it each single time.

Commit where the problem happens

a0d07fb5807ad55c8ccfdfc9a6d9ae3c62b9d211

What platforms do you use to access the UI ?

Windows

What browsers do you use to access the UI ?

Microsoft Edge

Command Line Arguments

--xformers --opt-channelslast --vae-path D:\WORK\conda_envs\automatic\stable-diffusion-webui\models\VAE\vae-ft-mse-840000-ema-pruned.pt --gfpgan-model GFPGANv1.4.pth

List of extensions

image

Console logs

This isn't relevant to this particular problem.

Additional information

No response

MrKuenning commented 1 year ago

Related https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/8511

missionfloyd commented 1 year ago

Why do you have each one is its own subfolder? It's okay to rename them.

MrKuenning commented 1 year ago

Mine are just in category folders and it is still super spaced out and hard to read. Also a LOT more scrolling.

2023-03-25 18_30_59-Jump joy local - Remote Desktop Connection

missionfloyd commented 1 year ago

8943 fixes the spacing.

UI designers these days are obsessed with whitespace.

f-rank commented 1 year ago

Why do you have each one is its own subfolder? It's okay to rename them.

Because sub folders with extra keywords, when needed, are the fastest way to remember. It's okay.