anapnoe / stable-diffusion-webui-ux

Stable Diffusion web UI UX
GNU Affero General Public License v3.0
978 stars 59 forks source link

[Bug]: Extra network cards overflow when folder buttons have multiple rows #55

Closed SirVeggie closed 1 year ago

SirVeggie commented 1 year ago

Is there an existing issue for this?

What happened?

image

Last row of cards is missing the names since the container overflows off-screen:

image

Modified this line in css to fix it temporarily:

image

Steps to reproduce the problem

-

What should have happened?

Network card container size needs to take into account the size of folder button container.

Commit where the problem happens

2ac62ce241396939c387a221e20b0a7a8c399b6f

What platforms do you use to access the UI ?

Windows

What browsers do you use to access the UI ?

Brave

Command Line Arguments

-

List of extensions

-

Console logs

-

Additional information

No response

anapnoe commented 1 year ago

I am thinking to change the buttons with a dropdown selection that will not affect the layout height an other solution would be to get the height from subdirs and subtract from the grid overflow div but this will not work if you have 100 categories and the height of the subdirs is greater than the grid overflow

SirVeggie commented 1 year ago

Dropdown would work, but it's much slower to navigate than buttons so I'm quite hesitant to use it for those edge cases. Most optimal would be to have it switch dynamically depending on the amount, but that seems a bit too ambitious.

Another option could be to include the buttons into the scroll area itself, so they can scroll out of view.

anapnoe commented 1 year ago

yep I will do it this way I will include the buttons to the overflow container