Skip to content

Commit

Permalink
recommend place infite scroll list at bottom (#30852)
Browse files Browse the repository at this point in the history
  • Loading branch information
zhouyx committed Oct 26, 2020
1 parent 988f809 commit c237ff1
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions extensions/amp-list/amp-list.md
Original file line number Diff line number Diff line change
Expand Up @@ -345,6 +345,8 @@ For working examples, please see [test/manual/amp-list/infinite-scroll-1.amp.htm

[tip type="important"]

When using `<amp-list>` infinite scroll, content placed below the component may not be accessible, and it is recommended to place the infinite scroll content at the bottom of the document.

When using `<amp-list>` infinite scroll in conjunction with `<amp-analytics>` scroll triggers, it is recommended to make use of the `useInitialPageSize` property of `<amp-analytics>` to get a more accurate measurement of the scroll position that ignores the height changes caused by `<amp-list>`.

Without `useInitialPageSize`, the `100%` scroll trigger point might never fire as more documents get loaded. Note that this will also ignore the size changes caused by other extensions (such as expanding embedded content) so some scroll events might fire prematurely instead.
Expand Down

0 comments on commit c237ff1

Please sign in to comment.