Skip to content

[Feature] CldVideoPlayer autoplay should respect browser standard prop of autoplay without captilization #319

@colbyfayock

Description

@colbyfayock

Feature Request

Is your feature request related to a problem? Please describe.

The browser standard video tag uses autoplay without capitalization. The CldVideoPlayer should have the same.

Describe the solution you'd like

autoplay="always" should result in autoplayMode="always" on the playerOptions object.

further, we should support the autoplay prop being passed as a boolean, so:

<CldVideoPlayer autoplay
<CldVideoPlayer autoplay={true}

This however shoudl act as if it were the standard autoplay attribute, which would add autoplay: true to the playerOptions object.

For backwards compatability, autoPlay should still be supported, but pass its value to autoplay

To sum up:

  • <CldVideoPlayer autoplay => playerOptions.autoplay = true
  • <CldVideoPlayer autoplay={true} => playerOptions.autoplay = true
  • <CldVideoPlayer autoplay={false} => playerOptions.autoplay = false
  • <CldVideoPlayer autoplay="always" => playerOptions.autoplayMode = 'always'
  • <CldVideoPlayer autoplay="never" => playerOptions.autoplayMode = 'never'
  • <CldVideoPlayer autoPlay="always" => playerOptions.autoplayMode = 'always'

Booleans get passed to autoplay and strings get passed to autoplayMode

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions