Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge v7.8.3 into Dev #2

Closed
wants to merge 6 commits into from
Closed

Merge v7.8.3 into Dev #2

wants to merge 6 commits into from

Conversation

DispatchCommit
Copy link

The fix in videojs#6297 doesn't work where the child to insert before is an element rather than a component, e.g. the video element.
Check if the child to insert before is an element, as well as checking if it has an el_

Fixes videojs#6640

Co-authored-by: Steve Heffernan git@heff.me

Co-authored-by: mister-ben git@misterben.me
Co-authored-by: Soroush Chehresa s1996ch@gmail.com
Co-authored-by: Gary Katsevman git@gkatsev.com
Co-authored-by: Owen Edwards owen.r.edwards@gmail.com
Co-authored-by: Steve Heffernan git@heff.me

Description

Please describe the change as necessary.
If it's a feature or enhancement please be as detailed as possible.
If it's a bug fix, please link the issue that it fixes or describe the bug in as much detail.

Specific Changes proposed

Please list the specific changes involved in this pull request.

Requirements Checklist

  • Feature implemented / Bug fixed
  • If necessary, more likely in a feature request than a bug fix
    • Change has been verified in an actual browser (Chome, Firefox, IE)
    • Unit Tests updated or fixed
    • Docs/guides updated
    • Example created (starter template on JSBin)
  • Reviewed by Two Core Contributors

mister-ben and others added 6 commits May 26, 2020 17:00
…ideojs#6644)

The fix in videojs#6297 doesn't work where the child to insert before is an element rather than a component, e.g. the video element.
Check if the child to insert before is an element, as well as checking if it has an el_
…' keyword (videojs#6603)

* Update descritpion of video.js in the package.json file, and add 'hls' keyword

* Update package.json

Co-authored-by: Steve Heffernan <git@heff.me>
* fix: addChild with index should allow for children that are elements (videojs#6644)

The fix in videojs#6297 doesn't work where the child to insert before is an element rather than a component, e.g. the video element.
Check if the child to insert before is an element, as well as checking if it has an el_

* docs(README): Update CDN version urls (videojs#6658)

* fix(fs): don't set player element css props on native fullscreen (videojs#6673)

Fixes videojs#6640

* Update description of video.js in the package.json file, and add 'hls' keyword (videojs#6603)

* Update descritpion of video.js in the package.json file, and add 'hls' keyword

* Update package.json

Co-authored-by: Steve Heffernan <git@heff.me>

Co-authored-by: mister-ben <git@misterben.me>
Co-authored-by: Soroush Chehresa <s1996ch@gmail.com>
Co-authored-by: Gary Katsevman <git@gkatsev.com>
Co-authored-by: Owen Edwards <owen.r.edwards@gmail.com>
Co-authored-by: Steve Heffernan <git@heff.me>
@DispatchCommit
Copy link
Author

too many conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants