Skip to content

Enhance the embed block to support all WP_oEmbed embeds #816

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
notnownikki merged 16 commits into from
Jun 2, 2017
Merged

Enhance the embed block to support all WP_oEmbed embeds #816

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
f9d17bc
Updated the embed block to use the oembed proxy API
notnownikki May 31, 2017
2723e86
Lint fixes
notnownikki May 31, 2017
d071080
Fix tests
notnownikki May 31, 2017
6fea1e8
Fix initial render of the embed block when loading a saved post
notnownikki May 31, 2017
830b743
Polish the embed placeholder a bit.
jasmussen Jun 1, 2017
b99e42c
Fix video and "other" ambiguation.
jasmussen Jun 1, 2017
c55973f
Address a couple of PR review items
jasmussen Jun 1, 2017
540c43c
Fixed components import
notnownikki Jun 1, 2017
42172c0
Adds wpApiSettings to the globals in .eslintrc.json
notnownikki Jun 1, 2017
ea6ff87
Deal with the component unmounting while a fetch is ongoing
notnownikki Jun 1, 2017
19f8477
Comment explaining the purpose of HtmlEmbed component
notnownikki Jun 1, 2017
f09a47f
Remove default prop in HtmlEmbed
notnownikki Jun 1, 2017
a235861
New script element creation tidied up
notnownikki Jun 1, 2017
d934958
Lint fix
notnownikki Jun 1, 2017
316fb7d
Fix typo
notnownikki Jun 1, 2017
58acca1
Fixed comment style
notnownikki Jun 2, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fixed comment style
  • Loading branch information
@notnownikki
notnownikki committed Jun 2, 2017
commit 58acca1bf0a9dded9b7648a1a1e5541688a86f61
11 changes: 5 additions & 6 deletions components/html-embed/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
/***
* When embedding HTML from the WP oEmbed proxy, we need to insert it
* into a div and make sure any scripts get run. This component takes
* HTML and puts it into a div element, and creates and adds new script
* elements so all scripts get run as expected.
*/
// When embedding HTML from the WP oEmbed proxy, we need to insert it
// into a div and make sure any scripts get run. This component takes
// HTML and puts it into a div element, and creates and adds new script
// elements so all scripts get run as expected.

export default class HtmlEmbed extends wp.element.Component {

componentDidMount() {
Expand Down