Bug 1325594 - Prevent controls being tabbed inside video control. r=mikedeboer draft
authorRay Lin <ralin@mozilla.com>
Wed, 04 Jan 2017 17:18:42 +0800
changeset 456124 525182f5c39bfb912bc18d32d1bc27814cccbc32
parent 455971 a2741dd43eeae54f4dd7423bd832a761481c56ce
child 541154 604ed546f69f8a9bc5d9e3780b9bc7a0a5015e5c
push id40413
push userbmo:ralin@mozilla.com
push dateThu, 05 Jan 2017 03:21:33 +0000
reviewersmikedeboer
bugs1325594
milestone53.0a1
Bug 1325594 - Prevent controls being tabbed inside video control. r=mikedeboer MozReview-Commit-ID: cYSumQ084S
toolkit/content/widgets/videocontrols.xml
--- a/toolkit/content/widgets/videocontrols.xml
+++ b/toolkit/content/widgets/videocontrols.xml
@@ -146,38 +146,40 @@
         <div class="controlsSpacerStack" aria-hideen="true">
           <div anonid="controlsSpacer" class="controlsSpacer stackItem" role="none"></div>
           <div anonid="clickToPlay" class="clickToPlay" hidden="true"></div>
         </div>
         <div anonid="controlBar" class="controlBar" hidden="true">
           <button anonid="playButton"
                   class="playButton"
                   playlabel="&playButton.playLabel;"
-                  pauselabel="&playButton.pauseLabel;"/>
+                  pauselabel="&playButton.pauseLabel;"
+                  tabindex="-1"/>
           <div anonid="scrubberStack" class="scrubberStack progressContainer" role="none">
             <div class="progressBackgroundBar stackItem" role="none">
               <div class="progressStack" role="none">
-                <progress anonid="bufferBar" class="bufferBar" value="0" max="100"></progress>
-                <progress anonid="progressBar" class="progressBar" value="0" max="100"></progress>
+                <progress anonid="bufferBar" class="bufferBar" value="0" max="100" tabindex="-1"></progress>
+                <progress anonid="progressBar" class="progressBar" value="0" max="100" tabindex="-1"></progress>
               </div>
             </div>
-            <input type="range" anonid="scrubber" class="scrubber"/>
+            <input type="range" anonid="scrubber" class="scrubber" tabindex="-1"/>
           </div>
           <span anonid="positionLabel" class="positionLabel" role="presentation"></span>
           <span anonid="durationLabel" class="durationLabel" role="presentation"></span>
           <span anonid="positionDurationBox" class="positionDurationBox" aria-hidden="true">
             &positionAndDuration.nameFormat;
           </span>
           <div anonid="controlBarSpacer" class="controlBarSpacer" hidden="true" role="none"></div>
           <button anonid="muteButton"
                   class="muteButton"
                   mutelabel="&muteButton.muteLabel;"
-                  unmutelabel="&muteButton.unmuteLabel;"/>
+                  unmutelabel="&muteButton.unmuteLabel;"
+                  tabindex="-1"/>
           <div anonid="volumeStack" class="volumeStack progressContainer" role="none">
-            <input type="range" anonid="volumeControl" class="volumeControl" min="0" max="100" step="1"/>
+            <input type="range" anonid="volumeControl" class="volumeControl" min="0" max="100" step="1" tabindex="-1"/>
           </div>
           <button anonid="closedCaptionButton" class="closedCaptionButton"/>
           <button anonid="fullscreenButton"
                   class="fullscreenButton"
                   enterfullscreenlabel="&fullscreenButton.enterfullscreenlabel;"
                   exitfullscreenlabel="&fullscreenButton.exitfullscreenlabel;"/>
         </div>
         <div anonid="textTrackList" class="textTrackList" hidden="true" offlabel="&closedCaption.off;"></div>