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

Select prefix padding is broken when it has multiple attribute #2167

Closed
AlexandreBonaventure opened this issue Sep 11, 2024 · 1 comment · Fixed by #2174 · May be fixed by TaylorBundy/swiper#3
Closed

Select prefix padding is broken when it has multiple attribute #2167

AlexandreBonaventure opened this issue Sep 11, 2024 · 1 comment · Fixed by #2174 · May be fixed by TaylorBundy/swiper#3
Labels
bug Things that aren't working right in the library.

Comments

@AlexandreBonaventure
Copy link

Describe the bug

Hi, I noticed the select prefix padding is off when the select is multiple. I hope this is a supported use-case.
I suppose the CSS rule was there for a reason :) (maybe to accomodate the option tags?) but it seems to conflict with this use case

Thanks.

ps: If you provide some guidance, I'll be happy to PR

To Reproduce

https://codepen.io/alexandrebonaventure/pen/XWLGGMJ

Screenshots

image The CSS rule causing the issue: image

Browser / OS

  • OS: Mac
  • Browser: Chrome
  • Browser version: 128
@AlexandreBonaventure AlexandreBonaventure added the bug Things that aren't working right in the library. label Sep 11, 2024
claviska added a commit that referenced this issue Sep 18, 2024
@claviska
Copy link
Member

Thanks for reporting this. It's been fixed in #2174 and will be available in the next Shoelace release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Things that aren't working right in the library.
Projects
None yet
2 participants