[wp-trac] [WordPress Trac] #20814: Add a couple other caveats for catching tweets for oEmbed
WordPress Trac
wp-trac at lists.automattic.com
Sat Jun 2 00:01:15 UTC 2012
#20814: Add a couple other caveats for catching tweets for oEmbed
-------------------------------------+----------------------
Reporter: georgestephanis | Owner:
Type: defect (bug) | Status: closed
Priority: normal | Milestone:
Component: General | Version: 3.4
Severity: normal | Resolution: invalid
Keywords: needs-patch ux-feedback |
-------------------------------------+----------------------
Changes (by nacin):
* status: new => closed
* resolution: => invalid
* milestone: Awaiting Review =>
Comment:
I don't know if these should catch as a tweet. They're not — they're a
link to an image. I am pretty sure if the image has two photos, you'd find
that /photo/2 shows different content than /photo/1. It's up to Twitter to
decide whether to serve up an oEmbed response for this.
We *could* limit the regex to avoid catching this kind of URL, but that
ends up with the exact same user experience — nothing shows up. So by
allowing the regex to catch this URL, we are forward compatible with a
potential change on Twitter's side to serve up a response.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/20814#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list