ampproject / amphtml

The AMP web component framework.
https://amp.dev
Apache License 2.0
14.89k stars 3.89k forks source link

🌸 Cherry-pick request for #22552 into #22426 (Approved) #22568

Closed dreamofabear closed 5 years ago

dreamofabear commented 5 years ago

Cherry-pick request

Issue PR Production? RC? Release issue
#22552 #22559 YES YES #22426

Why does this issue meet the cherry-pick criteria?

In some cases, causes amp-list to not render. Specifically, amp-list that are mutated via [src] binding with local data (not URLs) where the amp-list element has a zero size (width or height == 0).

Mini-postmortem

Summary

22227 deferred rendering of amp-list [src] mutations with local data until layout. This caused amp-list elements that wouldn't be laid out normally (e.g. zero size) to never render.

Users affected Impact
Pages that use amp-list[src] mutations with local data that have a zero size. Upon interaction via AMP.setState, the amp-list may not appear due to its size being zero.

Root Causes

Action Items

Action Item Type Owner PR #
Add regression test Prevent @choumx #22559

Lessons Learned

Things that went well

Things that went wrong


/cc @ampproject/wg-approvers @ampproject/cherry-pick-approvers

dvoytenko commented 5 years ago

Approved.