aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDylan Barrell <dylan@barrell.com>2013-10-26 11:31:07 -0400
committerScott González <scott.gonzalez@gmail.com>2013-12-19 10:36:30 -0500
commitb9e438d07c370ac2d4b198048feb6b6922469f70 (patch)
tree053e2b484fb30ef9ac5ed0579a169edcc6ce2c80
parentc9042b960f5c7b83a52e3b5c79dab2fed03f27e1 (diff)
downloadjquery-ui-b9e438d07c370ac2d4b198048feb6b6922469f70.tar.gz
jquery-ui-b9e438d07c370ac2d4b198048feb6b6922469f70.zip
Tooltip: Improve accessibility by adding content to an aria-live div
Fixes #9610 Closes gh-1118
-rw-r--r--tests/unit/tooltip/tooltip_core.js28
-rw-r--r--tests/unit/tooltip/tooltip_options.js9
-rw-r--r--ui/jquery.ui.tooltip.js25
3 files changed, 54 insertions, 8 deletions
diff --git a/tests/unit/tooltip/tooltip_core.js b/tests/unit/tooltip/tooltip_core.js
index c3568bffc..710444b44 100644
--- a/tests/unit/tooltip/tooltip_core.js
+++ b/tests/unit/tooltip/tooltip_core.js
@@ -21,27 +21,47 @@ test( "markup structure", function() {
});
test( "accessibility", function() {
- expect( 5 );
+ expect( 15 );
- var tooltipId,
- tooltip,
- element = $( "#multiple-describedby" ).tooltip();
+ var tooltipId, tooltip,
+ element = $( "#multiple-describedby" ).tooltip(),
+ liveRegion = element.tooltip( "instance" ).liveRegion;
+ equal( liveRegion.find( ">div" ).length, 0 );
+ equal( liveRegion.attr( "role" ), "log" );
+ equal( liveRegion.attr( "aria-live" ), "assertive" );
+ equal( liveRegion.attr( "aria-relevant" ), "additions" );
element.tooltip( "open" );
tooltipId = element.data( "ui-tooltip-id" );
tooltip = $( "#" + tooltipId );
equal( tooltip.attr( "role" ), "tooltip", "role" );
equal( element.attr( "aria-describedby" ), "fixture-span " + tooltipId,
"multiple describedby when open" );
+
// strictEqual to distinguish between .removeAttr( "title" ) and .attr( "title", "" )
// support: jQuery <1.6.2
// support: IE <8
// We should use strictEqual( ..., undefined ) when dropping jQuery 1.6.1 support (or IE6/7)
ok( !element.attr( "title" ), "no title when open" );
+ equal( liveRegion.children().length, 1 );
+ equal( liveRegion.children().last().html(), "..." );
element.tooltip( "close" );
equal( element.attr( "aria-describedby" ), "fixture-span",
"correct describedby when closed" );
equal( element.attr( "title" ), "...", "title restored when closed" );
+
+ element.tooltip( "open" );
+ equal( liveRegion.children().length, 2,
+ "After the second tooltip show, there should be two children" );
+ equal( liveRegion.children().filter( ":visible" ).length, 1,
+ "Only one of the children should be visible" );
+ ok( liveRegion.children().last().is( ":visible" ),
+ "Only the last child should be visible" );
+ element.tooltip( "close" );
+
+ element.tooltip( "destroy" );
+ equal( liveRegion.parent().length, 0,
+ "Tooltip liveregion element should be removed" );
});
test( "delegated removal", function() {
diff --git a/tests/unit/tooltip/tooltip_options.js b/tests/unit/tooltip/tooltip_options.js
index 01ac25040..bced8e860 100644
--- a/tests/unit/tooltip/tooltip_options.js
+++ b/tests/unit/tooltip/tooltip_options.js
@@ -41,13 +41,16 @@ test( "content: return string", function() {
});
test( "content: return jQuery", function() {
- expect( 1 );
+ expect( 2 );
var element = $( "#tooltipped1" ).tooltip({
content: function() {
- return $( "<div>" ).html( "cu<b>s</b>tomstring" );
+ return $( "<div id='unique'>" ).html( "cu<b id='bold'>s</b>tomstring" );
}
- }).tooltip( "open" );
+ }).tooltip( "open" ),
+ liveRegion = element.tooltip( "instance" ).liveRegion;
deepEqual( $( "#" + element.data( "ui-tooltip-id" ) ).text(), "customstring" );
+ equal( liveRegion.children().last().html(), "<div>cu<b>s</b>tomstring</div>",
+ "The accessibility live region will strip the ids but keep the structure" );
});
asyncTest( "content: sync + async callback", function() {
diff --git a/ui/jquery.ui.tooltip.js b/ui/jquery.ui.tooltip.js
index 9fde036a8..1ebe1a958 100644
--- a/ui/jquery.ui.tooltip.js
+++ b/ui/jquery.ui.tooltip.js
@@ -82,6 +82,16 @@ $.widget( "ui.tooltip", {
if ( this.options.disabled ) {
this._disable();
}
+
+ // Append the aria-live region so tooltips announce correctly
+ this.liveRegion = $( "<div>" )
+ .attr({
+ role: "log",
+ "aria-live": "assertive",
+ "aria-relevant": "additions"
+ })
+ .addClass( "ui-helper-hidden-accessible" )
+ .appendTo( this.document[ 0 ].body );
},
_setOption: function( key, value ) {
@@ -211,7 +221,7 @@ $.widget( "ui.tooltip", {
},
_open: function( event, target, content ) {
- var tooltip, events, delayedShow,
+ var tooltip, events, delayedShow, a11yContent,
positionOption = $.extend( {}, this.options.position );
if ( !content ) {
@@ -245,6 +255,18 @@ $.widget( "ui.tooltip", {
this._addDescribedBy( target, tooltip.attr( "id" ) );
tooltip.find( ".ui-tooltip-content" ).html( content );
+ // Support: Voiceover on OS X, JAWS on IE <= 9
+ // JAWS announces deletions even when aria-relevant="additions"
+ // Voiceover will sometimes re-read the entire log region's contents from the beginning
+ this.liveRegion.children().hide();
+ if ( content.clone ) {
+ a11yContent = content.clone();
+ a11yContent.removeAttr( "id" ).find( "[id]" ).removeAttr( "id" );
+ } else {
+ a11yContent = content;
+ }
+ $( "<div>" ).html( a11yContent ).appendTo( this.liveRegion );
+
function position( event ) {
positionOption.of = event;
if ( tooltip.is( ":hidden" ) ) {
@@ -394,6 +416,7 @@ $.widget( "ui.tooltip", {
element.removeData( "ui-tooltip-title" );
}
});
+ this.liveRegion.remove();
}
});