#841 Improve support for default embeds #889
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,65 @@ | ||
| <?php | ||
| /** | ||
| * Class AMP_Issuu_Embed_Handler | ||
| * | ||
| * @package AMP | ||
| * @since 0.7 | ||
| */ | ||
|
|
||
| /** | ||
| * Class AMP_Issuu_Embed_Handler | ||
| */ | ||
| class AMP_Issuu_Embed_Handler extends AMP_Base_Embed_Handler { | ||
| /** | ||
| * Regex matched to produce output amp-iframe. | ||
| * | ||
| * @const string | ||
| */ | ||
| const URL_PATTERN = '#https?://(www\.)?issuu\.com/.+/docs/.+#i'; | ||
|
|
||
| /** | ||
| * Register embed. | ||
| */ | ||
| public function register_embed() { | ||
| add_filter( 'embed_oembed_html', array( $this, 'filter_embed_oembed_html' ), 10, 3 ); | ||
| } | ||
|
|
||
| /** | ||
| * Unregister embed. | ||
| */ | ||
| public function unregister_embed() { | ||
| remove_filter( 'embed_oembed_html', array( $this, 'filter_embed_oembed_html' ), 10 ); | ||
| } | ||
|
|
||
| /** | ||
| * Filter oEmbed HTML for Meetup to prepare it for AMP. | ||
| * | ||
| * @param mixed $return The shortcode callback function to call. | ||
| * @param string $url The attempted embed URL. | ||
| * @param array $attr An array of shortcode attributes. | ||
| * @return string Embed. | ||
| */ | ||
| public function filter_embed_oembed_html( $return, $url, $attr ) { | ||
| $parsed_url = wp_parse_url( $url ); | ||
| if ( false !== strpos( $parsed_url['host'], 'issuu.com' ) ) { | ||
| if ( preg_match( '/width\s*:\s*(\d+)px/', $return, $matches ) ) { | ||
| $attr['width'] = $matches[1]; | ||
| } | ||
| if ( preg_match( '/height\s*:\s*(\d+)px/', $return, $matches ) ) { | ||
| $attr['height'] = $matches[1]; | ||
| } | ||
|
|
||
| $return = AMP_HTML_Utils::build_tag( | ||
| 'amp-iframe', | ||
| array( | ||
| 'width' => $attr['width'], | ||
| 'height' => $attr['height'], | ||
| 'src' => $url, | ||
| 'sandbox' => 'allow-scripts allow-same-origin', | ||
| ) | ||
| ); | ||
| } | ||
| return $return; | ||
| } | ||
| } | ||
|
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,52 @@ | ||
| <?php | ||
| /** | ||
| * Class AMP_Meetup_Embed_Handler | ||
| * | ||
| * @package AMP | ||
| * @since 0.7 | ||
| */ | ||
|
|
||
| /** | ||
| * Class AMP_Meetup_Embed_Handler | ||
| */ | ||
| class AMP_Meetup_Embed_Handler extends AMP_Base_Embed_Handler { | ||
|
|
||
| /** | ||
| * Regex matched. | ||
| * | ||
| * @const string | ||
| */ | ||
| const URL_PATTERN = '#https?://(www\.)?meetu(\.ps|p\.com)/.*#i'; | ||
|
|
||
| /** | ||
| * Register embed. | ||
| */ | ||
| public function register_embed() { | ||
| add_filter( 'embed_oembed_html', array( $this, 'filter_embed_oembed_html' ), 10, 2 ); | ||
| } | ||
|
|
||
| /** | ||
| * Unregister embed. | ||
| */ | ||
| public function unregister_embed() { | ||
| remove_filter( 'embed_oembed_html', array( $this, 'filter_embed_oembed_html' ), 10 ); | ||
| } | ||
|
|
||
| /** | ||
| * Filter oEmbed HTML for Meetup to prepare it for AMP. | ||
| * | ||
| * @param string $cache Cache for oEmbed. | ||
| * @param string $url Embed URL. | ||
| * @return string Embed. | ||
| */ | ||
| public function filter_embed_oembed_html( $cache, $url ) { | ||
| $parsed_url = wp_parse_url( $url ); | ||
| if ( false !== strpos( $parsed_url['host'], 'meetup.com' ) ) { | ||
|
|
||
| // Supply the width/height so that we don't have to make requests to look them up later. | ||
| $cache = str_replace( '<img ', '<img width="50" height="50" ', $cache ); | ||
| } | ||
| return $cache; | ||
| } | ||
| } | ||
|
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,73 @@ | ||
| <?php | ||
| /** | ||
| * Class AMP_Reddit_Embed_Handler | ||
| * | ||
| * @package AMP | ||
| * @since 0.7 | ||
| */ | ||
|
|
||
| /** | ||
| * Class AMP_Reddit_Embed_Handler | ||
| */ | ||
| class AMP_Reddit_Embed_Handler extends AMP_Base_Embed_Handler { | ||
| /** | ||
| * Regex matched to produce output amp-reddit. | ||
| * | ||
| * @const string | ||
| */ | ||
| const URL_PATTERN = '#https?://(www\.)?reddit\.com/r/[^/]+/comments/.*#i'; | ||
|
|
||
| /** | ||
| * Register embed. | ||
| */ | ||
| public function register_embed() { | ||
| wp_embed_register_handler( 'amp-reddit', self::URL_PATTERN, array( $this, 'oembed' ), -1 ); | ||
| } | ||
|
|
||
| /** | ||
| * Unregister embed. | ||
| */ | ||
| public function unregister_embed() { | ||
| wp_embed_unregister_handler( 'amp-reddit', -1 ); | ||
| } | ||
|
|
||
| /** | ||
| * Embed found with matching URL callback. | ||
| * | ||
| * @param array $matches URL regex matches. | ||
| * @param array $attr Additional parameters. | ||
| * @param array $url URL. | ||
| * @return string Embed. | ||
| */ | ||
| public function oembed( $matches, $attr, $url ) { | ||
| return $this->render( array( 'url' => $url ) ); | ||
| } | ||
|
|
||
| /** | ||
| * Output the Reddit amp element. | ||
| * | ||
| * @param array $args parameters used for output. | ||
| * @return string Rendered content. | ||
| */ | ||
| public function render( $args ) { | ||
| $args = wp_parse_args( $args, array( | ||
| 'url' => false, | ||
| ) ); | ||
|
|
||
| if ( empty( $args['url'] ) ) { | ||
| return ''; | ||
| } | ||
|
|
||
| return AMP_HTML_Utils::build_tag( | ||
| 'amp-reddit', | ||
| array( | ||
| 'layout' => 'responsive', | ||
| 'data-embedtype' => 'post', | ||
| 'width' => '100', | ||
| 'height' => '100', | ||
| 'data-src' => $args['url'], | ||
| ) | ||
| ); | ||
| } | ||
| } | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,74 @@ | ||
| <?php | ||
| /** | ||
| * Class AMP_Tumblr_Embed_Handler | ||
| * | ||
| * @package AMP | ||
| * @since 0.7 | ||
| */ | ||
|
|
||
| /** | ||
| * Class AMP_Tumblr_Embed_Handler | ||
|
There was a problem hiding this comment. Please add |
||
| */ | ||
| class AMP_Tumblr_Embed_Handler extends AMP_Base_Embed_Handler { | ||
| /** | ||
| * Regex matched to produce output amp-iframe. | ||
| * | ||
| * @const string | ||
| */ | ||
| const URL_PATTERN = '#https?://(.+)\.tumblr\.com/post/(.*)#i'; | ||
|
|
||
| /** | ||
| * Register embed. | ||
| */ | ||
| public function register_embed() { | ||
| wp_embed_register_handler( 'tumblr', self::URL_PATTERN, array( $this, 'oembed' ), -1 ); | ||
| } | ||
|
|
||
| /** | ||
| * Unregister embed. | ||
| */ | ||
| public function unregister_embed() { | ||
| wp_embed_unregister_handler( 'tumblr', -1 ); | ||
| } | ||
|
|
||
| /** | ||
| * Embed found with matching URL callback. | ||
| * | ||
| * @param array $matches URL regex matches. | ||
| * @param array $attr Additional parameters. | ||
| * @param array $url URL. | ||
| * @return string Embed. | ||
| */ | ||
| public function oembed( $matches, $attr, $url ) { | ||
| return $this->render( array( 'url' => $url ) ); | ||
| } | ||
|
|
||
| /** | ||
| * Output the Tumblr iframe. | ||
| * | ||
| * @param array $args parameters used for output. | ||
| * @return string Rendered content. | ||
| */ | ||
| public function render( $args ) { | ||
| $args = wp_parse_args( $args, array( | ||
| 'url' => false, | ||
| ) ); | ||
|
|
||
| if ( empty( $args['url'] ) ) { | ||
| return ''; | ||
| } | ||
|
|
||
| // Must enforce https for amp-iframe, but editors can supply either on desktop. | ||
| $args['url'] = str_replace( 'http://', 'https://', $args['url'] ); | ||
|
There was a problem hiding this comment. Unfortunately it seems this is not working fully. When attempting to embed https://ifpaintingscouldtext.tumblr.com/post/92003045635/grant-wood-american-gothic-1930 then Tumblr will redirect from HTTPS to HTTP for some reason. When serving a page over HTTPS, as they should be, then the iframe is blocked from rendering. |
||
|
|
||
| return AMP_HTML_Utils::build_tag( | ||
| 'amp-iframe', | ||
| array( | ||
| 'width' => $this->args['width'], | ||
| 'height' => $this->args['height'], | ||
| 'src' => $args['url'], | ||
| ) | ||
| ); | ||
| } | ||
| } | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add
@sincetags to the phpdoc.