Accessibility: Manage Rooms Page has Multiple Non-Unique Links when More than One Chat Room is Defined

Description

When multiple chat rooms are defined, the "Edit", "Delete", "Clear History", "Set as Default" links do not make sense when taken out of context when logged in as an instructor on the "Manage Rooms" page. The link text for links needs to make sense and describe what will happen when the link is activated. This is because many adaptive technologies (such as screen-readers) will pull together a list of all the links on a page and allow the user to select and activate them. If there are multiple "Delete" links, for instance, the user can't determine what will happen when a given "Delete" link is activated. Also, when they occur in between listed items, the low-vision/blind user can't use visual clues to determine if the link action refers to the chat room listed before, or the last room listed after, the link.

Suggested Fix:
Add hidden link text that adds name of the chat room as context to screen reader users. It is common practice in Sakai to use the CSS class "skip" to do this. For example:

HTML:

<div id="..." class="itemAction"><a ...>Edit Settings<span class="skip">: Main Chat Room.</span></a>

 

<a ...>Delete<span class="skip">: Main Chat Room.</span></a>

 

<a ...>Clear History<span class="skip">: Main Chat Room.</span></a>

 

<a ...>Set as Default<span class="skip">: Main Chat Room.</span></a>

Attachments

1

Activity

Matt Clare October 30, 2014 at 3:23 PM

Verified working with Chromevox on http://nightly2.sakaiproject.org:8082/. Thanks.

Noted, but not related, "Users in Chat" iframe on right in the initial chat area is a whole additional Sakai HTML page.

Hudson CI Server October 21, 2014 at 2:08 AM

UNSTABLE: Integrated in sakai-trunk-java-1.7 #485 (See http://builds.sakaiproject.org:8080/job/sakai-trunk-java-1.7/485/)
patch from G. Silverio for accessibility issue, inverting the link text parts to target:action (neal.caidin@apereo.org: rev 314681)

Neal Caidin October 20, 2014 at 8:58 PM

Committed with revision 314681 into trunk.

Neal Caidin October 20, 2014 at 3:04 PM

Cool. I see if we can get this patch added.

Gonzalo Silverio September 19, 2014 at 10:11 AM

I inverted the link text parts to

target: action

as this made more syntactical sense given the possible actions.

Fixed

Details

Priority

Fix versions

Components

Assignee

Reporter

Created November 23, 2010 at 11:08 AM
Updated October 30, 2014 at 3:23 PM
Resolved October 20, 2014 at 8:58 PM