Adding loading attribute for sl-menu-item

[mitchray]

Would it be possible to bring the sl-button loading attribute to the sl-menu-item?

I had a collection of buttons that was growing so I put them into an overflow sl-menu, but they lost their ability to show the loading spinner

I feel the sl-menu-item is for all intents and purposes a button so it makes sense for this behavior to carry over, thoughts?

[claviska]

When would you expect the menu item show a loading state? If it's unavailable when the menu opens, I would expect it to be disabled. If you're looking for it to show a loading state after it's selected, the menu would be closed by then so it wouldn't be visible.

[mitchray]

Indeed for a loading state after its selected, in an sl-dropdown with stay-open-on-select to allow multiple interactions

e.g. fetching a bunch of songs before playing them, or other promise based actions where it would be helpful to indicate a background task is underway

This is the behavior I'm hoping to replicate:

YPGd8VV28r.mp4

[mitchray]

I can take a stab at a PR for this in about a months time