-
Notifications
You must be signed in to change notification settings - Fork 384
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.
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
[2.1] Set amp-wp-unknown-size
class and object-fit=contain
for videos with unknown dimensions
#6578
Conversation
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.
LGTM .
@@ -4,7 +4,8 @@ | |||
* width such as in the post content so that the contents do not get stretched or cropped. | |||
* See <https://github.com/ampproject/amphtml/issues/21371#issuecomment-475443219>. | |||
*/ | |||
.amp-wp-enforced-sizes { | |||
.amp-wp-enforced-sizes, | |||
.amp-wp-unknown-size { |
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.
Good catch.
Codecov Report
@@ Coverage Diff @@
## 2.1 #6578 +/- ##
=========================================
Coverage 75.70% 75.70%
- Complexity 5918 5921 +3
=========================================
Files 237 237
Lines 17928 17931 +3
=========================================
+ Hits 13572 13575 +3
Misses 4356 4356
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Plugin builds for 807cf9b are ready 🛎️!
|
Cherry-picks #6576 (for #6571) into the 2.1 branch, with the one addition of 41f6fbe which came originally from #6518.