Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Adding video-wrapper for mobile fiendly preview of the youtube video #3171

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Oussamaosman02
Copy link

Before submitting a pull request, please make sure the following is done...

  1. Fork the repo and create your branch from master.
  2. If you've added code that should be tested, add tests!
  3. If you've changed APIs, update the documentation.
  4. Ensure that:
  • The test suite passes (npm test)
  • Your code lints (npm run lint) and passes Flow (npm run flow)
  • You have followed the testing guidelines
  1. If you haven't already, complete the CLA.

Please use the simple form below as a guideline for describing your pull request, and delete these instructions.

Thanks for contributing to Draft.js!


Summary

I added a ifr-wrapper class that wraps the iframe and h2 where is the youtube video. In a less than 560px screen, the youtube video continues pretending to have the 560px dimensions and thats disturbs the page making everithing to move to the left leaving a great withe space

Test Plan**

[...]

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

Successfully merging this pull request may close these issues.

2 participants