From 1810022d73edab98a57c5ab8c49d828bb311b400 Mon Sep 17 00:00:00 2001
From: jmclaus <jmc@edx.org>
Date: Fri, 18 Oct 2013 11:54:52 +0200
Subject: [PATCH] Caption functionality announced when tabbing to list

---
 common/lib/xmodule/xmodule/js/src/video/09_video_caption.js | 4 ++++
 lms/templates/video.html                                    | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/common/lib/xmodule/xmodule/js/src/video/09_video_caption.js b/common/lib/xmodule/xmodule/js/src/video/09_video_caption.js
index 668281b533..678c194f67 100644
--- a/common/lib/xmodule/xmodule/js/src/video/09_video_caption.js
+++ b/common/lib/xmodule/xmodule/js/src/video/09_video_caption.js
@@ -432,14 +432,18 @@ function () {
         // outline has to be drawn (tabbing) or not (mouse click).
         this.videoCaption.isMouseFocus = false;
 
+        // Set top and bottom spacing heigh and make sure they are taken out of
+        // the tabbing order.
         this.videoCaption.subtitlesEl
             .prepend(
                 $('<li class="spacing">')
                     .height(this.videoCaption.topSpacingHeight())
+                    .attr('tabindex', -1)
             )
             .append(
                 $('<li class="spacing">')
                     .height(this.videoCaption.bottomSpacingHeight())
+                    .attr('tabindex', -1)
             );
 
         this.videoCaption.rendered = true;
diff --git a/lms/templates/video.html b/lms/templates/video.html
index a32070432d..07855f1aae 100644
--- a/lms/templates/video.html
+++ b/lms/templates/video.html
@@ -85,7 +85,7 @@
             </section>
         </article>
 
-        <ol class="subtitles" tabindex="0" title="Captions"><li></li></ol>
+        <ol class="subtitles" tabindex="0" title="${_('Captions')}" role="group" aria-label="${_('Activating an item in this group will spool the video to the corresponding time point')}"><li></li></ol>
     </div>
 
     <div class="focus_grabber last"></div>