#42039 closed enhancement (fixed)
Widgets: Enable oEmbed support for Video widget
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.9 | Priority: | high |
| Severity: | normal | Version: | 4.8 |
| Component: | Widgets | Keywords: | needs-patch |
| Focuses: | Cc: |
Description (last modified by westonruter)
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)
@westonruter
8 weeks ago
#1
@westonruter
8 weeks ago
- Description modified (diff)
#2
@westonruter
8 weeks 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.
8 weeks ago
#4
@westonruter
8 weeks ago
- Priority changed from normal to high
#5
@timmydcrawford
7 weeks ago
@westonruter what is the timeline for the task / when is it needed by? Sorry kind of unplugged from the 4.9 calendar.
#6
@westonruter
7 weeks ago
@timmydcrawford beta1 is due Wednesday (in 2 days), so that would be the timeline.
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
7 weeks ago
This ticket was mentioned in Slack in #core by westonruter. View the logs.
7 weeks ago
#9
@westonruter
7 weeks ago
- Owner set to obenland
- Status changed from new to assigned
#10
@obenland
7 weeks ago
@westonruter @melchoyce What do you think of 42039.2.diff?
#13
@westonruter
7 weeks ago
- Resolution set to fixed
- Status changed from assigned to closed
We can follow-up with fixing the rendering of the preview in a defect ticket.
#14
@westonruter
6 weeks ago
In 41827:
#15
@westonruter
5 weeks ago
In 41933:
Work in progress to get another contributor started