slFocus and slBlur events fire twice on slSelect component

Describe the bug When an slSelect component gains or loses focus, the slFocus and slBlur events are called twice.

To Reproduce Steps to reproduce the behavior:

1. On https://shoelace.style/components/select add an event listener to a sl-select element in the console, $0.addEventListener('slFocus', () => console.log('slFocus'))
2. Focus the sl-select by clicking on it
3. slFocus is shown twice in the console
4. Repeat for the slBlur event

Expected behavior The slFocus and slBlur events should just be fired once.

[claviska]

Fixed in b51978b

[Marvinified]

@claviska after further digging I found out that the slBlur & slFocus event handlers for sl-select are also linked to sl-input that is being used in the component which is why the handler is being called twice.

This means there would be no need to remove the event emmiter in sl-select as I suggested before.

I would suggest just detaching the handler from sl-input here👇🏻 so it doesn’t get emitted twice.

https://github.com/shoelace-style/shoelace/blob/84d4421575c530f9145bea88b77cf06591bcf698/src/components/select/select.tsx#L409-L410