[Bug]: Screen Readers doesnt pick up `sp-progressbar`

Rajdeepc commented 1 month ago

Code of conduct

[X] I agree to follow this project's code of conduct.

Impacted component(s)

Progress-bar

Expected behavior

On tab stop on sp-progressbar, screen readers should be able to pickup the element content

Actual behavior

No tab stop on sp-progressbar

Screenshots

No response

What browsers are you seeing the problem in?

Firefox, Chrome, Safari, Microsoft Edge

How can we reproduce this issue?

Go to https://opensource.adobe.com/spectrum-web-components/storybook/index.html?path=/story/progress-bar--label
Click on Tab
Open Voice Over Utility
try to focus on the Progressbar
See it cannot be focussed

Sample code that illustrates the problem

protected override firstUpdated(changes: PropertyValues): void {
    super.firstUpdated(changes);
    if (!this.hasAttribute('role')) {
        this.setAttribute('role', 'progressbar');
    }
   // this needs to be added
    if (!this.hasAttribute('tabindex')) {
        this.setAttribute('tabindex', '0');
    }
}

Logs taken while reproducing problem

No response

nikkimk commented 1 month ago

@Rajdeepc this should be done with aria-live not a tabindex as progressbars are not interactive