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

Implement minimum penalty for ads #2561

Closed
cramforce opened this issue Mar 11, 2016 · 3 comments
Closed

Implement minimum penalty for ads #2561

cramforce opened this issue Mar 11, 2016 · 3 comments

Comments

@cramforce
Copy link
Member

We currently get hit by a special case (I believe) where if the first viewport contains no resources and the first resource on the page is an ad, then we load that ad immediately. This can strongly impact experience during swiping between articles.

I think we may want to put in an additional penalty that only applies at the very beginning of a page lifecycle.

CC @ericfs @dvoytenko @jridgewell

@dvoytenko
Copy link
Contributor

This is in the visible state?

@aprigogin
Copy link

I think so, yes. We ended up delaying placing amp into the visible state
for 1 second. That greatly improved fast swiping through viewer.

On Fri, Mar 11, 2016 at 4:23 PM, Dima Voytenko notifications@github.com
wrote:

This is in the visible state?


Reply to this email directly or view it on GitHub
#2561 (comment)
.

=G

@cramforce
Copy link
Member Author

Dima: Could you take this on? @jridgewell might also be great for you!

@jridgewell jridgewell assigned jridgewell and unassigned dvoytenko Mar 21, 2016
@rudygalfi rudygalfi added this to the Sprint 2016-03-31 [current] milestone Mar 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants