[wp-trac] [WordPress Trac] #37466: Add oEmbed support for Office Mix

WordPress Trac noreply at wordpress.org
Tue Jul 26 09:31:44 UTC 2016


#37466: Add oEmbed support for Office Mix
-------------------------+------------------------------
 Reporter:  GunGeekATX   |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Embeds       |     Version:
 Severity:  normal       |  Resolution:
 Keywords:               |     Focuses:
-------------------------+------------------------------

Comment (by swissspidy):

 Huh, I think that's the first time I've heard of OfficeMix. Looks like
 their support for oEmbed is fairly new and not even WordPress.com supports
 that provider.

 We have a certain standard for oEmbed providers in core. In order to add a
 new one to the existing whitelist, they must:

 * be well-established, popular, and mainstream services,
 * properly and fully implement the oEmbed specification,
 * and clearly be a trusted provider.

 See https://make.wordpress.org/core/handbook/contribute/design-decisions
 /#whitelisting-oembed-providers for a more complete checklist.

 Not respecting the maxwidth and maxheight parameters is clearly a bummer
 (see also #34737), so it would be good to be able to talk to their
 developers.

 It's worth noting that OfficeMix embeds kinda already work in WordPress
 thanks to oEmbed discovery and OfficeMix using an `<iframe>`. I only
 encountered a few problems:

 * The iframe tries to add a WebTrends tracking cookie, which it can't
 because of the sandboxing
 * It tries to load some JavaScript which doesn't exist
 * It also runs some deprecated features according to the console log.

 @GunGeekATX Thanks for your patch! Note that the table of oEmbed providers
 would need to be updated as well in the according DocBlock.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/37466#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list