{"action":"created","issue":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/819","labels_url":"https://api.github.com/repos/ampproject/amphtml/issues/819/labels{/name}","comments_url":"https://api.github.com/repos/ampproject/amphtml/issues/819/comments","events_url":"https://api.github.com/repos/ampproject/amphtml/issues/819/events","html_url":"https://github.com/ampproject/amphtml/pull/819","id":114435928,"number":819,"title":"[RFC] Tune heuristic when to load ads.","user":{"login":"cramforce","id":89679,"avatar_url":"https://avatars.githubusercontent.com/u/89679?v=3","gravatar_id":"","url":"https://api.github.com/users/cramforce","html_url":"https://github.com/cramforce","followers_url":"https://api.github.com/users/cramforce/followers","following_url":"https://api.github.com/users/cramforce/following{/other_user}","gists_url":"https://api.github.com/users/cramforce/gists{/gist_id}","starred_url":"https://api.github.com/users/cramforce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cramforce/subscriptions","organizations_url":"https://api.github.com/users/cramforce/orgs","repos_url":"https://api.github.com/users/cramforce/repos","events_url":"https://api.github.com/users/cramforce/events{/privacy}","received_events_url":"https://api.github.com/users/cramforce/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"dvoytenko","id":726049,"avatar_url":"https://avatars.githubusercontent.com/u/726049?v=3","gravatar_id":"","url":"https://api.github.com/users/dvoytenko","html_url":"https://github.com/dvoytenko","followers_url":"https://api.github.com/users/dvoytenko/followers","following_url":"https://api.github.com/users/dvoytenko/following{/other_user}","gists_url":"https://api.github.com/users/dvoytenko/gists{/gist_id}","starred_url":"https://api.github.com/users/dvoytenko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dvoytenko/subscriptions","organizations_url":"https://api.github.com/users/dvoytenko/orgs","repos_url":"https://api.github.com/users/dvoytenko/repos","events_url":"https://api.github.com/users/dvoytenko/events{/privacy}","received_events_url":"https://api.github.com/users/dvoytenko/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-10-31T21:38:17Z","updated_at":"2015-10-31T21:46:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ampproject/amphtml/pulls/819","html_url":"https://github.com/ampproject/amphtml/pull/819","diff_url":"https://github.com/ampproject/amphtml/pull/819.diff","patch_url":"https://github.com/ampproject/amphtml/pull/819.patch"},"body":"Instead of only loading them when they are in viewport, we also load\r\nthem if they are outside the viewport:\r\n\r\n- if the user has ever scrolled (So we don't interfere with e.g. swiping through articles).\r\n- we haven't started loading another ad in the last second.\r\n\r\nIn practice this should load a large percentage of ads before they enter the viewport\r\nwhile interference with scrolling is kept to a similar value as with the old heuristic."},"comment":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/comments/152775156","html_url":"https://github.com/ampproject/amphtml/pull/819#issuecomment-152775156","issue_url":"https://api.github.com/repos/ampproject/amphtml/issues/819","id":152775156,"user":{"login":"ampsauce","id":15166626,"avatar_url":"https://avatars.githubusercontent.com/u/15166626?v=3","gravatar_id":"","url":"https://api.github.com/users/ampsauce","html_url":"https://github.com/ampsauce","followers_url":"https://api.github.com/users/ampsauce/followers","following_url":"https://api.github.com/users/ampsauce/following{/other_user}","gists_url":"https://api.github.com/users/ampsauce/gists{/gist_id}","starred_url":"https://api.github.com/users/ampsauce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ampsauce/subscriptions","organizations_url":"https://api.github.com/users/ampsauce/orgs","repos_url":"https://api.github.com/users/ampsauce/repos","events_url":"https://api.github.com/users/ampsauce/events{/privacy}","received_events_url":"https://api.github.com/users/ampsauce/received_events","type":"User","site_admin":false},"created_at":"2015-10-31T21:46:38Z","updated_at":"2015-10-31T21:46:38Z","body":"\n**Tests failed.** Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: **BUSTED**\n\nCommit: 363e144f4449eacca67e4173e6b9084ad0a38bef\nBuild details: https://travis-ci.org/ampsauce/amphtml/builds/88571641\n\n(*Please note that this is a [fully automated](https://github.com/twbs/savage) comment.*)\n      "}}