#42039 closed enhancement (fixed)
Widgets: Enable oEmbed support for Video widget
Reported by: | westonruter | Owned by: | obenland |
---|---|---|---|
Milestone: | 4.9 | Priority: | high |
Severity: | normal | Version: | 4.8 |
Component: | Widgets | Keywords: | needs-patch |
Focuses: | Cc: |
Description (last modified by )
With #34115 now resolved, we can now support oEmbeds in the video widget instead of limiting the widget to just playing what is supported by MediaElement.js. See previously #40935 where the video widget was explicitly limited to support YouTube and Vimeo since these were the only hosted video providers that MediaElement.js supported directly without needing to resort to oEmbed.
See #39994 for the introduction of the video widget.
Note also that oEmbeds are now supported in the rich Text widget as well, so there may need to be some changes done for that widget in particular to apply the same inject_video_max_width_style
logic that the Video widget is applying now to the Video shortcode output.
Attachments (3)
Change History (18)
#2
@
3 years ago
@timmydcrawford Is this something you'd be interested in picking back up to round out your work on the Video widget?
This ticket was mentioned in Slack in #core-media by westonruter. View the logs.
3 years ago
#5
@
3 years ago
@westonruter what is the timeline for the task / when is it needed by? Sorry kind of unplugged from the 4.9 calendar.
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
3 years ago
This ticket was mentioned in Slack in #core by westonruter. View the logs.
3 years ago
#10
@
3 years ago
@westonruter @melchoyce What do you think of 42039.2.diff?
Work in progress to get another contributor started