aria-setsize and aria-posinset incorrectly set on Styles drop-down

[damo]

The items in the Styles drop-down do not have their correct aria-setsize and aria-posinset attributes set.

For example, when the selection is on a default paragraph and the Styles drop-down is displayed, the first item in the list has:
aria-posinset=5 and aria-setsize=21

These values should be:
aria-posinset=1 and aria-setsize=17

[j.swiderski]

But this set has 4 extra elements like "Image on the right" or "Borderless Table".

Yes they don't quite fit if you look that set has mainly elements from styles dropdown but is it causing any problems?

[damo]

@j.swiderski The extra elements you mention are present in the DOM but not displayed in the list. The problem for screen reader users is that when entering the list and focusing on the first element, the screen reader will read "option 5 of 21" when in fact, it is option 1 of 17, because the other elements are hidden and not reachable. This gives the wrong impression that there are available options in the list before the first element.

Each time the list is displayed these attributes need to update to reflect the actual state of the list.

Although, the way that the combo is implemented, aria-posinset and aria-setsize might not be required at all. The list of options does not dynamically change its contents as the user navigates it and all available options are rendered at once. See the WAI-ARIA spec for further details: ​http://www.w3.org/TR/wai-aria/states_and_properties#aria-posinset and ​http://www.w3.org/TR/wai-aria/states_and_properties#aria-setsize.

[j.swiderski]

Didn't pay attention to that "5 of 21" when testing. Sorry.

Reproducible from CKEditor 3.2

[garry.yao]

These two attributes are presented for guiding JAWS on old IEs, since not usable right now, let's see if screen reader behaviors well without it.

[fredck]

Ok, it makes sense.

[garry.yao]

Fixed with [7367].

[garry.yao]

The same change is missed on the color panel, post fixed with [7443].