1
0
mirror of https://github.com/videojs/video.js.git synced 2025-01-31 11:26:45 +02:00

Only rely on selectedIndex to set select option.

Firefox recently added selectedOptions support, however, unlike chrome
and other browsers, they made it read-only and have it throw an error.
In my testing (via http://jsfiddle.net/vwvdywf9/), on various browsers and platforms, just using selectedIndex was sufficient to change the selected option.

closes #1877
This commit is contained in:
Gary Katsevman 2015-02-18 12:10:13 -05:00 committed by heff
parent 8b0966c1b4
commit 14c94561a9
2 changed files with 1 additions and 4 deletions

View File

@ -3,6 +3,7 @@ CHANGELOG
## HEAD (Unreleased)
* @gkatsev fixed the track list reference while switching techs that use emulated tracks ([view](https://github.com/videojs/video.js/pull/1874))
* @gkatsev fixed a Firefox error with the captions settings select menu options ([view](https://github.com/videojs/video.js/pull/1877))
--------------------

View File

@ -163,10 +163,6 @@
}
}
if (target.selectedOptions) {
target.selectedOptions[0] = option;
}
target.selectedIndex = i;
}