[wp-trac] [WordPress Trac] #20814: Add a couple other caveats for catching tweets for oEmbed

[WordPress Trac]

#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