Twenty Twenty: video resize functionality also impacts other elements on the page

[jeherve]

Twenty Twenty bundles a functionality named "Intrinsic Ratio Embeds", allowing videos (iframe, object, video) to be automatically resized on demand.

This is practical, but can be problematic for some elements on the page that may not behave like a video, or that may not be in the post content at all.

While plugins can use the .intrinsic-ignore CSS class to avoid being impacted by this, I wonder if we could be a bit more specific within the theme, and only target videos inside the post content. This would avoid conflicting with plugins adding iFrames and / or videos outside of the post content, like in widgets.

[jeherve]

Only target videos inside post content

[lancewillett]

+1 for limiting the scope of this JavaScript script to only look for objects in post content.

[arcio]

Please fix this, not all iframes are videos.

[ianbelanger]

This functionality was added in the theme before we decided to add this add_theme_support( 'responsive-embeds' );. It may no longer be needed. I am going to test this to see if we can remove the whole Intrinsic Ratio Embeds function. Other testing would be appreciated. Thanks for bringing this up @jeherve.

[aidvu]

+1 to limiting the scope of elements the function looks at.

@ianbelanger what more needs to be done here?

[karmatosed]

Moving this to a future release as it has a patch and agreed approach. It now needs testing to check after some time has passed.