Skip to content
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

[Master feature] Loading state improvements for amp-list #14063

Open
ericlindley-g opened this Issue Mar 16, 2018 · 2 comments

Comments

@ericlindley-g
Copy link
Collaborator

ericlindley-g commented Mar 16, 2018

amp-list is associated with multiple loading states, particularly when integrated w/ with amp-bind & infinite scroll. Some of these states can already be configured by developers and get a default loading state in order to communicate status to users. This feature is to ensure that more of these states are configurable by developers and get good default states to the extent possible

Issues:
• Content not loaded (initial page load)
• src changed by amp-bind, content not yet loaded
• content to be appended via button or scroll-trigger, not yet loaded

@ericlindley-g ericlindley-g added this to the Prioritized FRs milestone Mar 16, 2018

@ericlindley-g ericlindley-g self-assigned this Mar 16, 2018

@ericlindley-g ericlindley-g added this to Feature Backlog in AMP HTML Project Roadmap via automation Mar 16, 2018

@ericlindley-g ericlindley-g changed the title [Master feature] [Master feature] Loading state improvements for amp-list Mar 16, 2018

@ericlindley-g ericlindley-g moved this from Feature Backlog to In Progress in AMP HTML Project Roadmap Mar 16, 2018

@ampprojectbot

This comment has been minimized.

Copy link
Collaborator

ampprojectbot commented Jun 5, 2018

This issue doesn't have a category which makes it harder for us to keep track of it. @ericlindley-g Please add an appropriate category.

@ampprojectbot

This comment has been minimized.

Copy link
Collaborator

ampprojectbot commented Feb 13, 2019

This issue hasn't been updated in awhile. @aghassemi Do you have any updates?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.