During the installation process of Drupal 7 installation progress is displayed as both a percentage and as a description of the task number and task being completed. This information is not automatically read for screen-reader users.

It is recommended to use an ARIA live region so that screen-reader users can be automatically alerted to the progress of the Drupal installation.

For an example see http://accessibleajax.clcworld.net/simple/live_polite.htm

For more information see http://wiki.codetalks.org/wiki/index.php/Set_of_ARIA_Test_Cases#aria-live

CommentFileSizeAuthor
#13 installation-progress-aria-513946-2.patch943 bytesmgifford
#3 installation-progress-aria-513946-1.patch781 bytesEverett Zufelt
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Everett Zufelt’s picture

Everett Zufelt CreditAttribution: Everett Zufelt commented
Title: Ehance installation progress by adding an ARIA live region » Enhance installation progress by adding an ARIA live region

Corrected spelling

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Issue tags: +Accessibility

Adding accessibility tag.

+1

Everett Zufelt’s picture

Everett Zufelt CreditAttribution: Everett Zufelt commented
Status: Active » Needs review
FileSize
installation-progress-aria-513946-1.patch781 bytes

This patch modifies misc/progress.js to add the aria-live attribute to the div container for the installation progress. The attribute is added as aria-live="polite".

This change provides updates to browsers and assistive technology which support aria when the installation progress has changed. In the case of a screen-reader the installation progress is then spoken aloud.

Tested with JAWS 10.0.1154 / FF 3.5.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

Patch applies nicely.

Looks like you're just adding 'aria-live="polite"' so I don't see any reason not to bring this into core.

Everett Zufelt’s picture

Everett Zufelt CreditAttribution: Everett Zufelt commented

Tested this patch today with JAWS10 / IE8 and Orca 2.22.2 / IceWeasel (Firefox) 3.0.6. Seems to be working well.

Also of interest blog article on screen-reader support for ARIA live regions.

Cliff’s picture

Cliff CreditAttribution: Cliff commented

Subscribe. And +1. (Two things one should be able to do without appearing to have commented.)

mattyoung’s picture

mattyoung CreditAttribution: mattyoung commented

.

Status: Needs review » Needs work

The last submitted patch failed testing.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Status: Needs work » Needs review

Bad bot. Ran just fine on a fresh install of the cvs for me.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Status: Needs review » Reviewed & tested by the community

This is a simple patch that will help blind users install Drupal easier.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Needs work

Could we document what this property is for in the source code? Looking at that I'd have no idea what that was for. I'd also immediately want to change 'polite' to 'rude' and see what happened. ;)

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

Good comment! I think aria-live="rude" would just be ignored by the screen reader. However, the possible settings are (at this point) just:

off/polite/assertive where off is the default.

  // The WAI-ARIA setting aria-live="polite" will announce changes after users 
  // have completed their current activity and not interrupt the screen reader
  this.element = $('<div class="progress" aria-live="polite"></div>').attr('id', id);

Some related links:
http://wiki.codetalks.org/wiki/index.php/Set_of_ARIA_Test_Cases
http://www.w3.org/WAI/PF/aria-practices/

If this documentation fits the requirements for javascript & clarity I'll re-roll the patch.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Status: Needs work » Needs review
FileSize
installation-progress-aria-513946-2.patch943 bytes

adding comment as per above to the patch.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Status: Needs review » Reviewed & tested by the community

Setting this back to rtbc.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Cool, thanks! Committed to HEAD.

Status: Fixed » Closed (fixed)
Issue tags: -Accessibility

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Accessibility

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.