[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