Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#21635 closed enhancement (fixed)

Add slideshare oEmbed support

Reported by: tyrun's profile Tyrun Owned by: nacin's profile nacin
Milestone: 3.5 Priority: normal
Severity: normal Version:
Component: Embeds Keywords: has-patch
Focuses: Cc:

Description

Adding speakerdeck and slideshare oEmbed support

Attachments (1)

speakerdeck-slideshare-oembed.diff (869 bytes) - added by Tyrun 12 years ago.

Download all attachments as: .zip

Change History (15)

#1 @Tyrun
12 years ago

#21632 thanks for the inspiration and getting the discussion started!!

#2 @bradparbs
12 years ago

  • Cc brad@… added

#3 @SergeyBiryukov
12 years ago

  • Milestone changed from Awaiting Review to 3.5

#4 @scribu
12 years ago

Speakerdeck is rather new, no?

#5 @SergeyBiryukov
12 years ago

Moved primarily for SlideShare.

#6 @DrewAPicture
12 years ago

+1 for adding SlideShare.

-1 for Speaker Deck. At barely a year old, I'm not sure its usage is widespread enough to justify adding core support just yet. There's already a plugin in the repo, Speaker Deck Embed with < 300 downloads.

#7 @Tyrun
12 years ago

Yes it is young, but many people are using it.

Additionally I think that providing support for services that makes 'magic' happen that just makes it easy for end users is a good thing.

I agree with Jaquith's thoughts on #21632, that the process of knowing that it can be done, knowing to look for a plugin to do it and then installing the plugin is too much for most people. At the cost of an additional element in an array I think it's definitely worth it.

#8 @scribu
12 years ago

It's not just "the cost of an additional element in an array". Each element in the array is another regex that the content of each post needs to go through, on each page load.

#9 @Tyrun
12 years ago

I see, I did not think of that. I still think having it available is a good thing to have.

Also the regex is only run on post save/update, correct?

#10 @scribu
12 years ago

No, it's run on post display.

#11 @tyrun
12 years ago

Ok - so for 3.5 should we just remove speakerdeck and just add support for slideshare?

#12 @nacin
12 years ago

  • Summary changed from Add speakerdeck and slideshare oEmbed support to Add slideshare oEmbed support

Slideshare is in, as long as its oEmbed endpoint is sane. It should be checked out for obeying sane maxheight and maxwidth values.

#13 @nacin
12 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [22003]:

Add soundcloud and slideshare to the oEmbed providers list. props bradparbs, ericwahlforss, wonderboymusic. fixes #15734, #21635.

#14 @tyrun
12 years ago

Hey @nacin, think I could possible get added to the contributors list since this was my ticket? It's my first ticket and I was really hoping to make the contributors list!

Also - does anyone have any idea how to change your handle on trac? I'd prefer to be aaronholbrook instead of my old handle of tyrun.

Note: See TracTickets for help on using tickets.