Skip to content

Update amphtml to 1905021827420#2297

Merged
westonruter merged 1 commit intodevelopfrom
update/amphtml-1905021827420
May 13, 2019
Merged

Update amphtml to 1905021827420#2297
westonruter merged 1 commit intodevelopfrom
update/amphtml-1905021827420

Conversation

@westonruter
Copy link
Copy Markdown
Member

Previously #2129.

  • Add object-fit and object-position to amp-img, amp-anim, and amp-video.
  • Add support for amp-link-rewriter.
  • Upgrade amp-experiment from 0.1 to 1.0.
  • Remove requirements for amp-addthis data attributes.

* Add `object-fit` and `object-position` to `amp-img`, `amp-anim`, and `amp-video`.
* Add support for `amp-link-rewriter`.
* Upgrade `amp-experiment` from `0.1` to `1.0`.
* Remove requirements for `amp-addthis` data attributes.
@westonruter westonruter added this to the v1.2 milestone May 13, 2019
@westonruter westonruter requested a review from amedina May 13, 2019 23:20
@googlebot googlebot added the cla: yes Signed the Google CLA label May 13, 2019
Copy link
Copy Markdown
Member

@amedina amedina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it.

@westonruter westonruter merged commit f887c26 into develop May 13, 2019
@westonruter westonruter deleted the update/amphtml-1905021827420 branch May 13, 2019 23:39
'requires_usage' => 2,
'version' => array(
'0.1',
'1.0',
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change broke a site. See https://wordpress.org/support/topic/google-optimize-integration-2/

The 1.0 version of amp-experiment is not stable yet (it's still itself experimental), necessitating sites to roll back to 0.1.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened issue to address: #4628

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Signed the Google CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants