From: Byron Jones <bjones@mozilla.com>
Date: Wed, 5 Mar 2014 14:45:38 +0000 (+0800)
Subject: Bug 977137: Comment and Preview tabs need accessibility markup
X-Git-Tag: bugzilla-4.5.3~44
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4e55e090eed7b6564c203caee535634c09a56920;p=thirdparty%2Fbugzilla.git

Bug 977137: Comment and Preview tabs need accessibility markup
r=dkl, a=justdave
---

diff --git a/js/field.js b/js/field.js
index 670c70a86c..51fd1fe86e 100644
--- a/js/field.js
+++ b/js/field.js
@@ -994,16 +994,22 @@ function show_comment_preview(bug_id) {
     var Dom = YAHOO.util.Dom;
     var comment = document.getElementById('comment');
     var preview = document.getElementById('comment_preview');
+
     if (!comment || !preview) return;
     if (Dom.hasClass('comment_preview_tab', 'active_comment_tab')) return;
 
     preview.style.width = (comment.clientWidth - 4) + 'px';
     preview.style.height = comment.offsetHeight + 'px';
 
+    var comment_tab = document.getElementById('comment_tab');
     Dom.addClass(comment, 'bz_default_hidden');
-    Dom.removeClass('comment_tab', 'active_comment_tab');
+    Dom.removeClass(comment_tab, 'active_comment_tab');
+    comment_tab.setAttribute('aria-selected', 'false');
+
+    var preview_tab = document.getElementById('comment_preview_tab');
     Dom.removeClass(preview, 'bz_default_hidden');
-    Dom.addClass('comment_preview_tab', 'active_comment_tab');
+    Dom.addClass(preview_tab, 'active_comment_tab');
+    preview_tab.setAttribute('aria-selected', 'true');
 
     Dom.addClass('comment_preview_error', 'bz_default_hidden');
 
@@ -1054,8 +1060,13 @@ function show_comment_edit() {
     if (!comment || !preview) return;
     if (YAHOO.util.Dom.hasClass(comment, 'active_comment_tab')) return;
 
+    var preview_tab = document.getElementById('comment_preview_tab');
     YAHOO.util.Dom.addClass(preview, 'bz_default_hidden');
-    YAHOO.util.Dom.removeClass('comment_preview_tab', 'active_comment_tab');
+    YAHOO.util.Dom.removeClass(preview_tab, 'active_comment_tab');
+    preview_tab.setAttribute('aria-selected', 'false');
+
+    var comment_tab = document.getElementById('comment_tab');
     YAHOO.util.Dom.removeClass(comment, 'bz_default_hidden');
-    YAHOO.util.Dom.addClass('comment_tab', 'active_comment_tab');
+    YAHOO.util.Dom.addClass(comment_tab, 'active_comment_tab');
+    comment_tab.setAttribute('aria-selected', 'true');
 }
diff --git a/template/en/default/bug/comment.html.tmpl b/template/en/default/bug/comment.html.tmpl
index e308019634..96cbb63ed1 100644
--- a/template/en/default/bug/comment.html.tmpl
+++ b/template/en/default/bug/comment.html.tmpl
@@ -13,10 +13,12 @@
   #%]
 
 [% IF feature_enabled('jsonrpc') %]
-  <div id="comment_tabs">
+  <div id="comment_tabs" role="tablist">
     <div id="comment_tab" class="comment_tab active_comment_tab"
+          role="tab" aria-selected="true"
           onclick="show_comment_edit()">Comment</div>
     <div id="comment_preview_tab" class="comment_tab"
+          role="tab" aria-selected="false"
           onclick="show_comment_preview([% bug.id FILTER none %])">Preview</div>
   </div>
 [% END %]