Video element overflows on summary pages when screen width is less than 700 px. about catchup HOT 6 CLOSED

Can you please link how the page is getting rendered? I want to see the html tree

from catchup.

Proposed solution: an easy fix would be to add width 100% to the styling of the video tag, the video will render according to the size available, taking 100% of it. Tried for multiple devices, it is working.

from catchup.

Proposed solution: an easy fix would be to add width 100% to the styling of the video tag, the video will render according to the size available, taking 100% of it. Tried for multiple devices, it is working.

Any updates? @HarshKapadia2 @tusharnankani @SirusCodes

from catchup.

Can you please link how the page is getting rendered? I want to see the html tree

Here:

We add a video to indicate that we forgot to take a picture during a particular CatchUp session, for example in the 74th CatchUp summary.

Proposed solution: an easy fix would be to add width 100% to the styling of the video tag, the video will render according to the size available, taking 100% of it. Tried for multiple devices, it is working.

Would you like to open a Pull Request? Feel free to make the changes in the /summary/static/css

from catchup.

Can you please link how the page is getting rendered? I want to see the html tree

Here:

We add a video to indicate that we forgot to take a picture during a particular CatchUp session, for example in the 74th CatchUp summary.

---

Proposed solution: an easy fix would be to add width 100% to the styling of the video tag, the video will render according to the size available, taking 100% of it. Tried for multiple devices, it is working.

Would you like to open a Pull Request? Feel free to make the changes in the /summary/static/css

Alright will do.

from catchup.

I have created the PR. Well sorry for the commit name, I had updated the commit name where it asks again for some reason it didn't show up.

from catchup.