[wp-trac] [WordPress Trac] #29126: Add Kickstarter oEmbed support

WordPress Trac noreply at wordpress.org
Thu Sep 18 20:18:00 UTC 2014


#29126: Add Kickstarter oEmbed support
-------------------------------------+------------------
 Reporter:  johnbillion              |       Owner:
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  4.1
Component:  Embeds                   |     Version:
 Severity:  minor                    |  Resolution:
 Keywords:  needs-patch 2nd-opinion  |     Focuses:
-------------------------------------+------------------

Comment (by helen):

 Observations about the endpoint:

 * Does not sanitize maxwidth. Seems to escape it, at least, but it's
 supposed to be an integer. Bad values set the height to 0.
 * Does not respect maxheight. Appears to ignore it completely. The "rich"
 type of oEmbed requires that maxwidth and maxheight be obeyed. Not a
 dealbreaker, but I have an easier time trusting providers who know the
 spec and follow it.

 Would also be curious as to how well it works in responsive themes.

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


More information about the wp-trac mailing list