Skip to content

Commit

Permalink
🤖 Merge PR DefinitelyTyped#61399 [video.js] Add missing player option…
Browse files Browse the repository at this point in the history
…: playsinline by @marcosarce

* Add missing player option: playsinline

videojs/video.js#4348
https://github.com/videojs/video.js/blob/v7.20.1/src/js/player.js#L1170

* Add 'playsinline' option to tests
  • Loading branch information
marcosarce authored and shlyren committed Aug 17, 2022
1 parent f7cf386 commit 0b8a70a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions types/video.js/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7134,6 +7134,7 @@ export interface VideoJsPlayerOptions extends videojs.ComponentOptions {
nativeControlsForTouch?: boolean | undefined;
notSupportedMessage?: string | undefined;
playbackRates?: number[] | undefined;
playsinline?: boolean | undefined;
noUITitleAttributes?: boolean | undefined;
plugins?: Partial<VideoJsPlayerPluginOptions> | undefined;
poster?: string | undefined;
Expand Down
1 change: 1 addition & 0 deletions types/video.js/video.js-tests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ const playerOptions: VideoJsPlayerOptions = {
nativeControlsForTouch: true,
notSupportedMessage: 'Oh no! :(',
playbackRates: [0.5, 1],
playsinline: false,
noUITitleAttributes: true,
plugins: {
myPlugin: {
Expand Down

0 comments on commit 0b8a70a

Please sign in to comment.