Invalid aria-posinset value

[kklein-celera]

Describe the bug

The carousel radio buttons have a aria-posinset attribute assigned, but the numbering starts at zero instead of 1. This is not fully compliant with the ARIA spec.
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Reference/Attributes/aria-posinset#values

Expected behavior

aria-posinset attributes values start at 1

Minimal reproducible example

No response

Add-on Version

3.0.0

Vaadin Version

24.7.6

Additional information

No response

[javier-godoy]

Confirmed. According to WAI-ARIA 1.2:

Authors MUST set the value for aria-posinset to an integer greater than or equal to 1

The attribute is assigned in the sources of l2t-paper-slider, which we have forked in our repository.

CarouselAddon/src/main/resources/META-INF/resources/frontend/paper-slider/l2t-paper-slider.js

Line 183 in f105525

<span tabindex="0" role="menuitemradio" aria-checked="false" class="slider__dot" aria-posinset\$="[[item]]"></span>

We must adjust how aria-posinset is used by the component:

CarouselAddon/src/main/resources/META-INF/resources/frontend/paper-slider/l2t-paper-slider.js

Line 482 in f105525

dotElems[i].setAttribute("aria-label", "Slide " + (parseInt(dotElems[i].getAttribute('aria-posinset')) + 1) + " selector");

CarouselAddon/src/main/resources/META-INF/resources/frontend/paper-slider/l2t-paper-slider.js

Line 464 in f105525

newPos = parseInt(dotElem.getAttribute('aria-posinset')),

[javier-godoy]

Thanks for reporting this issue. The fix has been released in version 3.0.1.