AztecEditor-iOS icon indicating copy to clipboard operation
AztecEditor-iOS copied to clipboard

Add support for Embeds

Open diegoreymendez opened this issue 8 years ago • 18 comments

As reported by @rachelmcr (by an original report from @kwonye):

I tested some oEmbeds and shortcodes in Aztec, and found that oEmbeds in Aztec for iOS don’t seem to be working. When I paste in a plain URL such as a YouTube or Twitter link, the URL appears in my post instead of converting to an embed. (The same URLs embedded when pasted into Aztec on Android or in a shortcode.)

We should make sure we support all the popular embeds.

This ticket requires a prior check against Aztec iOS for consistency.

diegoreymendez avatar Jul 06 '17 17:07 diegoreymendez

List of embeds supported by WordPress.org

We should create a priority for those, probably this should be base on the use of those.

@astralbodies any way to get stats of the use of embeds on WordPress.com? I suppose Twitter, YouTube and the likes to be very popular.

SergioEstevao avatar Jul 11 '17 11:07 SergioEstevao

Implementing this on aztec can be very complex and time consuming.

On a web view based solution we could simple implement it by mapping URL schemes to Iframes players provided by the different providers, or implement the Embed protocol to find the dynamically.

The simplest solution for us now could be detect this URL's and marked them with a special attachment type EmbedAttachmet that could be displayed in a different way.

When we tap on it we could then show a web view that shows the specific embed interface.

cc @diegoreymendez @astralbodies

SergioEstevao avatar Jul 11 '17 11:07 SergioEstevao

I just want to clarify that the main problem is that when you paste a link into Aztec, the link itself shows up on the published post (the oEmbed completely fails to work).

Although it would be amazing to have the oEmbed work in the editor itself, my main concern is that Aztec should support pasting in URLs that are converted to oEmbeds server side.

rachelmcr avatar Jul 11 '17 12:07 rachelmcr

A user brought this up after trying Aztec (internal ref: 205352-h). They expected Twitter embeds to work when created in the app.

rachelmcr avatar Jul 18 '17 18:07 rachelmcr

In light of the recent crashes / reports / upcoming improvements, this is still not high in the list of priorities.

That said, we'll revisit our priorities every sprint.

diegoreymendez avatar Jul 18 '17 21:07 diegoreymendez

Just wanted to note that this came up again in user feedback (ref: 206160-h), from a user who couldn't get Instagram and Twitter oEmbeds to work in their posts. (I want to note this so we can track user demand for this, but let me know if you'd prefer we track user feedback on this in a different way.)

rachelmcr avatar Jul 21 '17 10:07 rachelmcr

That's actually very useful @rachelmcr, and I think it will be valuable for us to decide what to tackle next.

diegoreymendez avatar Jul 21 '17 16:07 diegoreymendez

User expected YouTube embeds to work as they previously did, and wrote into support asking for help.

Internal ref: #211193-h

sarahblackstock avatar Aug 07 '17 15:08 sarahblackstock

Just a small note, we found some really critical issues we're currently dealing with, so this will still take a bit longer before we're able to tackle it.

Will keep everyone posted here.

diegoreymendez avatar Aug 07 '17 17:08 diegoreymendez

User request for Spotify:

1502621-zen

jkmassel avatar Oct 19 '18 14:10 jkmassel

Another request for YouTube in 1523542-zen.

thehenrybyrd avatar Oct 29 '18 08:10 thehenrybyrd

Another request for YouTube in 1533752-zen

donalirl avatar Oct 31 '18 22:10 donalirl

There's an initial effort to get pasting embeds working here: https://github.com/wordpress-mobile/AztecEditor-iOS/pull/1070

diegoreymendez avatar Nov 01 '18 15:11 diegoreymendez

I'm reopening this since there's more work we can do in terms of supporting embeds.

diegoreymendez avatar Nov 06 '18 18:11 diegoreymendez

I'm reopening this since there's more work we can do in terms of supporting embeds.

Thanks – this got autoclosed b/c of the merge 😅

jkmassel avatar Nov 06 '18 20:11 jkmassel

We have another user who reported this with Flickr embeds in 1572606-zen.

rezzap avatar Nov 17 '18 18:11 rezzap

Another request in 1607164-z, the user would like to embed Twitter statuses.

SiobhyB avatar Nov 30 '18 18:11 SiobhyB

Another request in 1619462-z, this time for Issuu embeds.

SiobhyB avatar Dec 06 '18 15:12 SiobhyB