{"action":"created","issue":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/3663","repository_url":"https://api.github.com/repos/ampproject/amphtml","labels_url":"https://api.github.com/repos/ampproject/amphtml/issues/3663/labels{/name}","comments_url":"https://api.github.com/repos/ampproject/amphtml/issues/3663/comments","events_url":"https://api.github.com/repos/ampproject/amphtml/issues/3663/events","html_url":"https://github.com/ampproject/amphtml/issues/3663","id":161276689,"number":3663,"title":"Iframe resize height adjustment","user":{"login":"pdufour","id":1239145,"avatar_url":"https://avatars.githubusercontent.com/u/1239145?v=3","gravatar_id":"","url":"https://api.github.com/users/pdufour","html_url":"https://github.com/pdufour","followers_url":"https://api.github.com/users/pdufour/followers","following_url":"https://api.github.com/users/pdufour/following{/other_user}","gists_url":"https://api.github.com/users/pdufour/gists{/gist_id}","starred_url":"https://api.github.com/users/pdufour/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pdufour/subscriptions","organizations_url":"https://api.github.com/users/pdufour/orgs","repos_url":"https://api.github.com/users/pdufour/repos","events_url":"https://api.github.com/users/pdufour/events{/privacy}","received_events_url":"https://api.github.com/users/pdufour/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-06-20T19:39:39Z","updated_at":"2016-06-22T21:51:21Z","closed_at":null,"body":"## Short description of your issue:\r\n\r\nI may be missing something, but I don't understand why there is a limit around making an iframe height smaller. I have a legitimate case where I have created a responsive ad that needs to adjust height at a couple of breakpoints.\r\n\r\nSource: https://github.com/ampproject/amphtml/blob/master/src/service/resources-impl.js#L757\r\n\r\n## How do we reproduce the issue?\r\n- Create an amp-ad\r\n- Fire an event to make the amp ad smaller than it currently is\r\n\r\n## What browsers are affected?\r\n\r\nAll browsers\r\n## Which AMP version is affected?\r\n\r\nv0\r\n"},"comment":{"url":"https://api.github.com/repos/ampproject/amphtml/issues/comments/227888056","html_url":"https://github.com/ampproject/amphtml/issues/3663#issuecomment-227888056","issue_url":"https://api.github.com/repos/ampproject/amphtml/issues/3663","id":227888056,"user":{"login":"jridgewell","id":112982,"avatar_url":"https://avatars.githubusercontent.com/u/112982?v=3","gravatar_id":"","url":"https://api.github.com/users/jridgewell","html_url":"https://github.com/jridgewell","followers_url":"https://api.github.com/users/jridgewell/followers","following_url":"https://api.github.com/users/jridgewell/following{/other_user}","gists_url":"https://api.github.com/users/jridgewell/gists{/gist_id}","starred_url":"https://api.github.com/users/jridgewell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jridgewell/subscriptions","organizations_url":"https://api.github.com/users/jridgewell/orgs","repos_url":"https://api.github.com/users/jridgewell/repos","events_url":"https://api.github.com/users/jridgewell/events{/privacy}","received_events_url":"https://api.github.com/users/jridgewell/received_events","type":"User","site_admin":false},"created_at":"2016-06-22T21:51:21Z","updated_at":"2016-06-22T21:51:21Z","body":"> The embed should receive back embed-size-denied event.\r\n\r\nNot when you're making it [smaller](https://github.com/ampproject/amphtml/blob/90837fe/src/service/resources-impl.js#L759-L761), that'll only happen when making it [bigger](https://github.com/ampproject/amphtml/blob/90837fe/src/service/resources-impl.js#L762-L766). I think this is a legitimate bug (not that we deny it, but that we don't notify you that we denied it)."}}
{"actor":{"display_login":"jridgewell"}}