[Bug][a11y]: AccordionItem should not have role="listitem"

[majornista]

Code of conduct

• I agree to follow this project's code of conduct.

Impacted component(s)

Accordion, AccordionItem

Expected behavior

sp-accordion-item should not expresses its WAI-ARIA role as a listitem.

Actual behavior

sp-accordion-item expresses its WAI-ARIA role as a listitem but the containing sp-accordion is not a list.

This seems to come from Spectrum-CSS, with:

spectrum-web-components/packages/accordion/src/spectrum-accordion-item.css

Line 45 in cd8c99f

display: list-item;

Coming from .spectrum-Accordion-item:

https://github.com/adobe/spectrum-css/blob/e9c58caecfcfa8dd71a25d582c2394ed120e6b5e/components/accordion/index.css#L53

Screenshots

No response

What browsers are you seeing the problem in?

Chrome, Safari

How can we reproduce this issue?

1. Go to https://opensource.adobe.com/spectrum-web-components/components/accordion/
2. Using Web Inspector tool, inspect the Accessibility properties for the first <sp-accordion-item> element in the first Accordion example.
3. Notice that in the Accessibility tree, the <sp-accordion-item> is identified as having role="listitem"
4. Using Web Inspector tool, inspect the Accessibility properties for the <sp-accordion> element that contains the <sp-accordion-item> in the first Accordion example.
5. Notice that in the Accessibility tree, the <sp-accordion> has no role defined.

Sample code that illustrates the problem

No response

Logs taken while reproducing problem

No response

Related issue: Spectrum CSS Issue #1884

[Rajdeepc]

Adding spectrum-css PR: adobe/spectrum-css#1789
Adding swc PR: #3300

[Rajdeepc]

The above PR is merged. waiting for the issue to be tested.

[Rajdeepc]

@pfulton @venkateshjo Can we mark this as done from our end too?
cc @jnjosh

[venkateshjo]

This issue is fixed in the latest swc.
we can close this issue.

[Rajdeepc]

Thanks @venkateshjo