HTML generated by UrlEmbedFilter
is inserted as is received from the external source. While there's nothing semantically or technically wrong with having an <IFRAME>
element next to <P>
elements, it would be useful to have more control of the generated output by wrapping it inside another <DIV>
element and throwing a few attributes to it (like CSS classes). I think it would be welcomed to have a few standard and predictable CSS selectors to style every kind of embedded content.
This also would facilitate implementing other features like Add option to make embeds responsive ✨ Add option to make embeds responsive Needs work (and others similar) as only a few lines of CSS targeting the wrapper element would be necessary.
Add a theming function inside UrlEmbedFilter::process()
to wrap the output generated by the external source inside a <DIV>
element.
Discuss the convenience of this approach and review a patch.
None.
None (I think).
None.
Needs review
1.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.