From 99ae63777e317183fa7fb5ac9fa0f35603dab337 Mon Sep 17 00:00:00 2001
From: Valera Rozuvan <valera@edx.org>
Date: Wed, 14 Aug 2013 11:51:15 +0300
Subject: [PATCH] Removed unnecessary tabindex = 0 from <a /> - they get it by
 default.

---
 .../xmodule/js/src/video/07_video_volume_control.js    |  3 ---
 lms/templates/video.html                               | 10 +++++-----
 2 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/common/lib/xmodule/xmodule/js/src/video/07_video_volume_control.js b/common/lib/xmodule/xmodule/js/src/video/07_video_volume_control.js
index 785401aa65..90154d2079 100644
--- a/common/lib/xmodule/xmodule/js/src/video/07_video_volume_control.js
+++ b/common/lib/xmodule/xmodule/js/src/video/07_video_volume_control.js
@@ -61,9 +61,6 @@ function () {
             slide: state.videoVolumeControl.onChange
         });
 
-        // Make sure that we can focus the actual volume slider while Tabing.
-        state.videoVolumeControl.volumeSliderEl.find('a').attr('tabindex', '0');
-
         state.videoVolumeControl.el.toggleClass('muted', state.videoVolumeControl.currentVolume === 0);
     }
 
diff --git a/lms/templates/video.html b/lms/templates/video.html
index 90b3e5b25f..43f36915a0 100644
--- a/lms/templates/video.html
+++ b/lms/templates/video.html
@@ -39,25 +39,25 @@
 
                 <div>
                     <ul class="vcr">
-                        <li><a class="video_control" href="#" tabindex="0" title="${_('Play')}"></a></li>
+                        <li><a class="video_control" href="#" title="${_('Play')}"></a></li>
                         <li><div class="vidtime">0:00 / 0:00</div></li>
                     </ul>
                     <div class="secondary-controls">
                         <div class="speeds">
-                            <a href="#" tabindex="0" title="Speeds">
+                            <a href="#" title="Speeds">
                                 <h3>${_('Speed')}</h3>
                                 <p class="active"></p>
                             </a>
                             <ol class="video_speeds"></ol>
                         </div>
                         <div class="volume">
-                            <a href="#" tabindex="0" title="Volume"></a>
+                            <a href="#" title="Volume"></a>
                             <div class="volume-slider-container">
                                 <div class="volume-slider"></div>
                             </div>
                         </div>
-                        <a href="#" class="add-fullscreen" tabindex="0" title="${_('Fill browser')}">${_('Fill browser')}</a>
-                        <a href="#" class="quality_control" tabindex="0" title="${_('HD')}">${_('HD')}</a>
+                        <a href="#" class="add-fullscreen" title="${_('Fill browser')}">${_('Fill browser')}</a>
+                        <a href="#" class="quality_control" title="${_('HD')}">${_('HD')}</a>
 
                         <a href="#" class="hide-subtitles" title="${_('Turn off captions')}">${_('Captions')}</a>
                     </div>