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

fix: Add poster size styles #8022

Merged
merged 1 commit into from
Nov 24, 2022
Merged

fix: Add poster size styles #8022

merged 1 commit into from
Nov 24, 2022

Conversation

mister-ben
Copy link
Contributor

@mister-ben mister-ben commented Nov 24, 2022

Description

Adds styling for the img el in the poster's picture so it scales correctly.

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 (Chrome, Firefox, IE)
    • Unit Tests updated or fixed
    • Docs/guides updated
    • Example created (starter template on JSBin)
  • Reviewed by Two Core Contributors

@mister-ben mister-ben changed the title fix: Poster size fix: Add poster size styles Nov 24, 2022
@codecov
Copy link

codecov bot commented Nov 24, 2022

Codecov Report

Merging #8022 (169fdc8) into main (84062c3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #8022   +/-   ##
=======================================
  Coverage   82.67%   82.67%           
=======================================
  Files         110      110           
  Lines        7342     7342           
  Branches     1772     1772           
=======================================
  Hits         6070     6070           
  Misses       1272     1272           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mister-ben mister-ben merged commit dff3dd4 into videojs:main Nov 24, 2022
edirub pushed a commit to edirub/video.js that referenced this pull request Jun 8, 2023
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.

2 participants