Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix padding on accordion/dropdown list elements #8943

Merged

Conversation

space-nuko
Copy link
Contributor

Describe what this pull request is trying to achieve.

Makes accordions have the padding they did before, while removing padding on dropdown list elements and adding selected highlight. Makes both closer to their appearance in old Gradio

Additional notes and description of your changes

Just some CSS additions, plus blocks layout elements get CSS classes like with IOComponents

Environment this was tested in

List the environment you have developed / tested this on. As per the contributing page, changes should be able to work on Windows out of the box.

  • OS: Windows
  • Browser: Chrome
  • Graphics card: NVIDIA RTX 3090

Screenshots or videos of your changes

Old:
2023-03-25 15_19_55-Stable Diffusion - Chromium
2023-03-19 23_51_32-Stable Diffusion - Chromium

New:
2023-03-25 15_51_38-Stable Diffusion - Chromium
2023-03-25 15_51_54-Stable Diffusion - Chromium

@hithereai
Copy link
Collaborator

Thanks for this ❤️

@oliverban
Copy link

Awesome, thanks! :)

@AUTOMATIC1111 AUTOMATIC1111 merged commit 3b5a3fa into AUTOMATIC1111:master Mar 27, 2023
@AUTOMATIC1111
Copy link
Owner

firefox_Le1OZ1rKRU

The selection looks pretty bad with white theme... Also gradio seems to show multiple items as selected erroneously.

@AUTOMATIC1111
Copy link
Owner

firefox_WxBfQaeClZ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants