Metadata
- Source
- FLUID-3227
- Type
- Bug
- Priority
- Minor
- Status
- Closed
- Resolution
- Fixed
- Assignee
- Alison Benjamin
- Reporter
- Colin Clark
- Created
2009-10-05T17:15:40.000-0400 - Updated
2014-03-03T14:22:36.360-0500 - Versions
-
- 1.1.1
- 1.1.2
- Fixed Versions
- N/A
- Component
-
- Undo
Description
In the Undo component, we create an ARIA live region like this, directly in the markup:
var markup = "<span class='flc-undo' aria-live='polite' aria-relevant='all'>" +
"<span class='flc-undo-undoContainer'>[<a href='#' class='flc-undo-undoControl'>undo</a>]</span>" +
"<span class='flc-undo-redoContainer'>[<a href='#' class='flc-undo-redoControl'>redo</a>]</span>" +
"</span>";
This markup is hard-baked to the newer Firefox 3-style syntax, which causes it not to work in Firefox 2. The unit tests also fail on Firefox 2 as a result.
We should always add ARIA attributes using jQuery rather than directly in markup, since it does this normalization for us.
Note that we're no longer formally testing against Firefox 2, so this isn't a huge deal but is something we should try to fix if we can find the time.
Attachments
Comments
-
Alison Benjamin commented
2009-10-22T16:15:56.000-0400 Adds ARIA using jquery. Works fine with JAWS, & unit tests pass in FF2 & FF3.
-
Justin Obara commented
2009-11-24T14:06:57.000-0500 Code Review Comments:
===================The live region itself doesn't work in FF 2, but the markup does appear to be correct. Did you experience this as well?
A bit off topic, what is your opinion on this. Since the undo/redo buttons will change the text of the content (the view mode text for inline edit), should these also be declared as live regions. Also should the undo and redo buttons describe what they are doing for example "undo edit".