From 3c1108c5dc89e6c48d4e75683e1e3cb9ff8e096e Mon Sep 17 00:00:00 2001 From: Kevin Litchfield Date: Wed, 15 Feb 2017 15:23:47 -0500 Subject: [PATCH] fix: Set MuteButton controlText correctly (#4056) --- src/js/control-bar/mute-toggle.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/js/control-bar/mute-toggle.js b/src/js/control-bar/mute-toggle.js index 157bb443d..694139da9 100644 --- a/src/js/control-bar/mute-toggle.js +++ b/src/js/control-bar/mute-toggle.js @@ -88,7 +88,8 @@ class MuteToggle extends Button { // Don't rewrite the button text if the actual text doesn't change. // This causes unnecessary and confusing information for screen reader users. // This check is needed because this function gets called every time the volume level is changed. - const toMute = this.player_.muted() ? 'Unmute' : 'Mute'; + const soundOff = this.player_.muted() || this.player_.volume() === 0; + const toMute = soundOff ? 'Unmute' : 'Mute'; if (this.controlText() !== toMute) { this.controlText(toMute);