search

microsoft / fast

The adaptive interface system for modern web experiences.
https://www.fast.design
Other
9.28k stars 595 forks source link

fix: fluent-search has a11y issue #6813

Closed vnbaaij closed 5 months ago

vnbaaij commented 1 year ago

The fluent-search component does not have any discernible text for the clear button. See https://github.com/microsoft/fluentui-blazor/issues/621 for more inforation

chrisdholt commented 1 year ago

The button should likely have aria-hidden set on it given it's not in the tabindex, although it's possible that this should really just be a slot and not bake in the button or glyph. That would be breaking, so the latter is likely the best path forward for vNext, while the archive might be mitigated with aria-hidden...

vnbaaij commented 1 year ago

I'll take a stab at adding the aria-hidden in a PR

janechu commented 5 months ago

Closing this as it appears to not be related to a PR in FAST.