r735 - in trunk: tests ui
Author: david.bolter
Date: Mon Sep 22 11:21:38 2008
New Revision: 735
Modified:
trunk/tests/core.js
trunk/ui/ui.core.js
Log:
Accept boolean values for aria state. (Thanks for catch Scott González)
Modified: trunk/tests/core.js
==============================================================================
--- trunk/tests/core.js (original)
+++ trunk/tests/core.js Mon Sep 22 11:21:38 2008
@@ -64,8 +64,8 @@
equals($('#aria').ariaRole(), 'tablist', 'role is tablist via ariaRole');
ok(!$('#aria').attr('expanded'), 'state expanded absent via attr');
ok(!$('#aria').ariaState('expanded'), 'state expanded absent via
ariaState');
-
equals($('#aria').ariaState('expanded', 'true').ariaState('expanded'), 'true', 'aria
expanded is true');
-
equals($('#aria').ariaState('expanded', 'false').ariaState('expanded'), 'false', 'aria
expanded is false');
+ equals($('#aria').ariaState('expanded',
true).ariaState('expanded'), 'true', 'aria expanded is true');
+ equals($('#aria').ariaState('expanded',
false).ariaState('expanded'), 'false', 'aria expanded is false');
});
})(jQuery);
Modified: trunk/ui/ui.core.js
==============================================================================
--- trunk/ui/ui.core.js (original)
+++ trunk/ui/ui.core.js Mon Sep 22 11:21:38 2008
@@ -454,7 +454,7 @@
ariaState : function(state, value) {
// setter?
- if (value)
+ if (value !== undefined)
return this.each(function(i, el) {
isFF2? el.setAttributeNS("http://www.w3.org/2005/07/aaa", "aaa:" +
state, value) :
$(el).attr("aria-" + state, value);