shoelace-style / shoelace

A collection of professionally designed, every day UI components built on Web standards. SHOELACE IS BECOMING WEB AWESOME 👇👇👇
https://webawesome.com
MIT License
12.93k stars 827 forks source link

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

Closed AlexandreBonaventure closed 1 month ago

AlexandreBonaventure commented 2 months ago

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

claviska commented 1 month ago

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