fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2011-12-22 10:18 am

[dw-free] Revamp /update

[commit: http://hg.dwscoalition.org/dw-free/rev/bf5f265c0ab8]

http://bugs.dwscoalition.org/show_bug.cgi?id=2524

Add aria-live to the collapsible target. Also ensure that the target has an
id, so that we can refer to this for the aria-controls.

Patch by [personal profile] fu.

Files modified:
  • htdocs/js/jquery.collapsible.js
--------------------------------------------------------------------------------
diff -r 8a4c11cb2a00 -r bf5f265c0ab8 htdocs/js/jquery.collapsible.js
--- a/htdocs/js/jquery.collapsible.js	Thu Dec 22 18:14:08 2011 +0800
+++ b/htdocs/js/jquery.collapsible.js	Thu Dec 22 18:18:34 2011 +0800
@@ -16,8 +16,11 @@
             return self;
 
         var opts = self.options;
+        self._target = $(opts.target, self.element)
         self._trigger = $trigger;
-        self._target = $(opts.target, self.element);
+
+        self._target.attr("aria-live","polite").filter(":not([id])").attr("id", "collapsibletarget_" + self.element.attr("id"))
+        self._trigger.attr("aria-controls", self._target.attr("id"))
         self.element.data("collapsibleid", $.proxy(opts.parseid, self.element)());
 
         $trigger.ultrafocus(function() {
--------------------------------------------------------------------------------