Help Videos Show Too Small
- This topic has 8 replies, 3 voices, and was last updated 4 years ago by .
Viewing 9 posts - 1 through 9 (of 9 total)
Viewing 9 posts - 1 through 9 (of 9 total)
- You must be logged in to reply to this topic.
Greetings,
On the BMCC OL, embedded YT videos show very small in the OL Help pages.
Example: https://openlab.bmcc.cuny.edu/blog/help/how-to-write-a-basic-post-with-the-block-editor-on-an-openlab-wordpress-site-video-tutorial/
We tried adding code, which worked for a few weeks, but then the code breaks, and the video stops embedding completely, like this: \https://openlab.bmcc.cuny.edu/blog/help/how-to-embed-a-google-doc-form-sheet-on-an-openlab-wordpress-site-video-tutorial-classic/
Here’s the code, which we usually just end up removing:
<div class=”videoWrapper ratio-16-9″>https://www.youtube.com/watch?v=jFC-O1xctlw</div>
We wonder if the help pages would embed YT videos properly if we had the block editor option. Would that be a possible feature to get? Or, is there another solution to properly embed videos?
Many thanks!
Hello – Thanks for sharing, Im going to take a peek at this, back at you soon.
Ryan
Thanks for the report.
It looks like the bug stems from the OpenLab theme stripping the width and height attributes in post content. I’ve filed a bug here: https://github.com/cuny-academic-commons/openlab-theme/issues/305
Thanks so much Ray!
Thanks so much for looking into this, @r-a-y and Boone.
We looked at Boone’s solve on github, and we’re following up because we’re not sure it’s the height/width issue.
We took a screencast showing that WordPress seems to remove the spaces around our div code/wrapper, over time. To solve the issue, we wonder if either WP could stop changing the spacing, or, we could get Gutenberg in the Help pages, if that’s possible.
Here is that video: https://bmcc-cuny.zoom.us/rec/play/ciNWQf8NeqjmieLXG0BYcXMmjtcZAA5YhMWAwdAZ6lbfSP4NVpG5yoFcJd8oafInPO2ckRhjezqH2kPL.cxr1oIxa5Jyebrq0?startTime=1632422806000
Thanks for your time!
If you’ve applied Boone’s latest changeset to the openlab-theme: https://github.com/cuny-academic-commons/openlab-theme/commit/37c8fa29c3532d337705e7bbdf1748ff04f65d97
You don’t need the videowrapper DIV anymore in your post editor because the width of the video should be 500px or higher. If the video’s width is not at least 500px, let us know. If you’d prefer a larger default size, a fix will be needed for the openlab-theme.
The only reason why you’d want to use your videowrapper DIV is I’m guessing you want responsive-sized videos if the screen is too small? I’m not too familiar with the videowrapper code you’re using. Is that from another plugin?
Thanks for the reply, @r-a-y! The solve did work, but, yes, as you asked about, we want the videos to be responsive. Do you have any ideas about why the div that we add around it gets removed?
Screenshots showing the video still too small without the div wrapper on PC (plus, on mobile, it’s too big…)
The problem is the openlab-theme
does not support responsive videos natively.
I’ve filed an issue here with some code to address this: https://github.com/cuny-academic-commons/commons-in-a-box/issues/371
Once the code is merged, you will not need to use your videowrapper DIV workaround.