Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

FLUID-4862: use aria-label instead of a hidden span

  • Loading branch information...
commit 5dd43f5b4ee9051afd99a2d8f4262e9121bb3755 1 parent 4f9d3b0
@acheetham acheetham authored
Showing with 1 addition and 6 deletions.
  1. +0 −5 html/videoPlayer_template.html
  2. +1 −1  js/ToggleButton.js
View
5 html/videoPlayer_template.html
@@ -13,12 +13,10 @@
<div class="fl-videoPlayer-controller-buttons">
<div class="fl-videoPlayer-controller-buttons-main">
<button type="button" class="flc-videoPlayer-play fl-videoPlayer-button">
- <span class="flc-videoPlayer-play-label fl-hidden-accessible"></span>
</button>
<div class="flc-videoPlayer-volumeContainer fl-videoPlayer-volumeContainer" title="Volume controls. Use up and down arrows to adjust volume, space or enter to mute">
<button type="button" class="flc-videoPlayer-mute fl-videoPlayer-button">
- <span class="flc-videoPlayer-mute-label fl-hidden-accessible"></span>
</button>
<div class="flc-videoPlayer-volumeControl"></div>
</div>
@@ -44,7 +42,6 @@
<div class="fl-videoPlayer-controller-buttons-secondary">
<div class="flc-videoPlayer-captionControls-container fl-videoPlayer-captions">
<button class="flc-videoPlayer-captions-button fl-videoPlayer-button">
- <span class="flc-videoPlayer-captions-label fl-hidden-accessible"></span>
</button>
<ul class="flc-videoPlayer-captions-languageMenu fl-videoPlayer-languageMenu">
<li class="flc-videoPlayer-language flc-videoPlayer-menuItem fl-videoPlayer-menuItem"></li>
@@ -54,7 +51,6 @@
<div class="flc-videoPlayer-transcriptControls-container fl-videoPlayer-transcript">
<button class="flc-videoPlayer-transcripts-button fl-videoPlayer-button">
- <span class="flc-videoPlayer-transcripts-label fl-hidden-accessible"></span>
</button>
<ul class="flc-videoPlayer-transcripts-languageMenu fl-videoPlayer-languageMenu">
<li class="flc-videoPlayer-language flc-videoPlayer-menuItem fl-videoPlayer-menuItem"></li>
@@ -63,7 +59,6 @@
</div>
<button id="fullscreen" type="button" class="flc-videoPlayer-fullscreen fl-videoPlayer-optional fl-videoPlayer-button">
- <span class="flc-videoPlayer-fullscreen-label fl-hidden-accessible"></span>
</button>
</div>
</div>
View
2  js/ToggleButton.js
@@ -82,7 +82,7 @@ https://github.com/fluid-project/infusion/raw/master/Infusion-LICENSE.txt
button.attr("aria-pressed", pressed);
var labelText = that.tooltipContentFunction(that);
- that.locate("label").text(labelText);
+ that.locate("button").attr("aria-label", labelText);
that.tooltip.updateContent(labelText);
};
Please sign in to comment.
Something went wrong with that request. Please try again.