fix: label attribute value should be set as text in listbox option

[yinonov]

🐛 Bug Report

setting label on option does not reflect to DOM

(notice option 2)

according to mdn

Label attribute is text for the label indicating the meaning of the option. If the label attribute isn't defined, its value is that of the element text content.

🤔 Expected Behavior

label should precede html content

(see first option)

[chrisdholt]

This will be difficult because we would need to override what is being projected into the default slot. I'm not sure how we could/would do that. While it's doable for the native element, I'm not sure we have the same ability to do this with how projection works. I certainly understand the desire to stay close here, but I'm not sure it's reasonably feasible or intuitive for this scenario.

[yinonov]

how about visually hiding the default slot and extract its textContent to populate into the template?
or some manipulation utilizing the slotted directive with filter?

can this go south?

I raised this ask a while back #6409

this is the discord correspondence

[chrisdholt]

@yinonov if you want to open a PR against the main branch I think we'd take something here as a contribution. I think you're on the stable release still which we can cherry-pick the change into.

[janechu]

Unfortunately @microsoft/fast-foundation is being deprecated, refer to #6955.