danielstorey / adapt-expose

An Adapt Framework component which displays panels of content hidden behind front covers. Click to expose the content below
GNU General Public License v3.0
5 stars 16 forks source link

Accessibility changes #21

Open canstudios-nicolaw opened 5 years ago

canstudios-nicolaw commented 5 years ago

This PR contains fixes to the accessibility of expose. There is now an 'Expose content' button at the bottom of each panel and content is read out by the screen reader when it is exposed.

This has been tested with the NVDA screen reader on Chrome, Firefox & IE11.

We are aware of one minor issue: While previewing a course in firefox and IE11 the screen reader will continuously read out the alt text of the last card on the page.

I am not sure where the best place is to put the 'Expose content' button text in the properties.schema. If you could please advise I will get this updated.

I am aware that currently, if the user does not specify an item title it is displayed as "?", which is not read out by screen readers. I'm not sure whether it would be sufficient to say this should be specified in order to make the course accessible or whether this needs an aria-label?