WordPress.org

Make WordPress Core

Opened 3 months ago

Closed 2 months ago

Last modified 2 months ago

#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 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)

42039.wip.diff (5.7 KB) - added by westonruter 3 months ago.
Work in progress to get another contributor started
42039.diff (5.4 KB) - added by obenland 2 months ago.
42039.2.diff (6.7 KB) - added by obenland 2 months ago.

Download all attachments as: .zip

Change History (18)

@westonruter
3 months ago

Work in progress to get another contributor started

#1 @westonruter
3 months ago

  • Description modified (diff)

#2 @westonruter
3 months 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 months ago

#4 @westonruter
3 months ago

  • Priority changed from normal to high

#5 @timmydcrawford
3 months 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
3 months 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.


2 months ago

This ticket was mentioned in Slack in #core by westonruter. View the logs.


2 months ago

#9 @westonruter
2 months ago

  • Owner set to obenland
  • Status changed from new to assigned

@obenland
2 months ago

@obenland
2 months ago

#10 @obenland
2 months ago

@westonruter @melchoyce What do you think of 42039.2.diff?

#11 @obenland
2 months ago

In 41759:

Widgets: Allow oEmbeds in video widget.

Opens up video embeds to all supported video oEmbed providers.

Props westonruter.
See #42039.

#12 @obenland
2 months ago

In 41765:

Widgets: Continue to use shortcode for some embeds

Renders YouTube and Vimeo embeds through WordPress' video shortcode.

See #42039.

#13 @westonruter
2 months 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
2 months ago

In 41827:

Widgets: Update video widget embed failure notice to indicate streams other than YouTube and Vimeo are allowed.

Amends [41759].
See #42039.

#15 @westonruter
2 months ago

In 41933:

Widgets: Harden logic for checking for valid media URLs.

  • Update deprecated isHostedVideo method to always return true since all oEmbeds are now supported.
  • Disable scanImage for non-image media widgets.
  • Ensure embed URL field element is matched from current media frame instance due to bug where media frames are not destroyed upon closing.
  • Update error message for invalid video URLs to ask user to check the URL for validity.

Props gk.loveweb, octalmage, westonruter.
See #42039, #40935.
Fixes #41052.

Note: See TracTickets for help on using tickets.