Skip to content

Commit

Permalink
fixed the Wrong caption icon 2 shaka-project#4358
Browse files Browse the repository at this point in the history
  • Loading branch information
joeyparrish authored and kartikeyshaurya committed Aug 4, 2022
1 parent b361948 commit 4c95296
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/build-and-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -86,10 +86,11 @@ jobs:
run: |
# Download Safari 14
# See also https://www.macupdate.com/app/mac/15675/apple-safari/old-versions
curl -Lv https://www.macupdate.com/action/download/63550 > safari-14.1.2.pkg
curl -Lv https://www.macupdate.com/action/download/62946 > Safari14.0CatalinaAuto.pkg.zip
# Install Safari 14 to homedir specifically.
installer -pkg safari-14.1.2.pkg -target CurrentUserHomeDirectory
unzip Safari14.0CatalinaAuto.pkg.zip
installer -pkg Safari14.0CatalinaAuto.pkg -target CurrentUserHomeDirectory
# Install a launcher that can execute a shell script to launch this
npm install karma-script-launcher --save-dev
Expand Down
4 changes: 2 additions & 2 deletions ui/text_selection.js
Original file line number Diff line number Diff line change
Expand Up @@ -111,11 +111,11 @@ shaka.ui.TextSelection = class extends shaka.ui.SettingsMenu {
onCaptionStateChange_() {
if (this.player.isTextTrackVisible()) {
this.icon.textContent =
shaka.ui.Enums.MaterialDesignIcons.CLOSED_CAPTIONS_OFF;
shaka.ui.Enums.MaterialDesignIcons.CLOSED_CAPTIONS;
this.button.ariaPressed = 'true';
} else {
this.icon.textContent =
shaka.ui.Enums.MaterialDesignIcons.CLOSED_CAPTIONS;
shaka.ui.Enums.MaterialDesignIcons.CLOSED_CAPTIONS_OFF;
this.button.ariaPressed = 'false';
}

Expand Down

0 comments on commit 4c95296

Please sign in to comment.