{"action":"created","issue":{"url":"https://api.github.com/repos/Automattic/wp-calypso/issues/23554","repository_url":"https://api.github.com/repos/Automattic/wp-calypso","labels_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/23554/labels{/name}","comments_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/23554/comments","events_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/23554/events","html_url":"https://github.com/Automattic/wp-calypso/pull/23554","id":307739579,"number":23554,"title":"Themes: Use photon on screenshots","user":{"login":"sirreal","id":841763,"avatar_url":"https://avatars2.githubusercontent.com/u/841763?v=4","gravatar_id":"","url":"https://api.github.com/users/sirreal","html_url":"https://github.com/sirreal","followers_url":"https://api.github.com/users/sirreal/followers","following_url":"https://api.github.com/users/sirreal/following{/other_user}","gists_url":"https://api.github.com/users/sirreal/gists{/gist_id}","starred_url":"https://api.github.com/users/sirreal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sirreal/subscriptions","organizations_url":"https://api.github.com/users/sirreal/orgs","repos_url":"https://api.github.com/users/sirreal/repos","events_url":"https://api.github.com/users/sirreal/events{/privacy}","received_events_url":"https://api.github.com/users/sirreal/received_events","type":"User","site_admin":false},"labels":[{"id":285009985,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/Themes","name":"Themes","color":"006b75","default":false},{"id":285010018,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/[Status]%20Needs%20Review","name":"[Status] Needs Review","color":"fbca04","default":false},{"id":285010109,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/[Type]%20Janitorial","name":"[Type] Janitorial","color":"fad8c7","default":false}],"state":"open","locked":false,"assignee":{"login":"sirreal","id":841763,"avatar_url":"https://avatars2.githubusercontent.com/u/841763?v=4","gravatar_id":"","url":"https://api.github.com/users/sirreal","html_url":"https://github.com/sirreal","followers_url":"https://api.github.com/users/sirreal/followers","following_url":"https://api.github.com/users/sirreal/following{/other_user}","gists_url":"https://api.github.com/users/sirreal/gists{/gist_id}","starred_url":"https://api.github.com/users/sirreal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sirreal/subscriptions","organizations_url":"https://api.github.com/users/sirreal/orgs","repos_url":"https://api.github.com/users/sirreal/repos","events_url":"https://api.github.com/users/sirreal/events{/privacy}","received_events_url":"https://api.github.com/users/sirreal/received_events","type":"User","site_admin":false},"assignees":[{"login":"sirreal","id":841763,"avatar_url":"https://avatars2.githubusercontent.com/u/841763?v=4","gravatar_id":"","url":"https://api.github.com/users/sirreal","html_url":"https://github.com/sirreal","followers_url":"https://api.github.com/users/sirreal/followers","following_url":"https://api.github.com/users/sirreal/following{/other_user}","gists_url":"https://api.github.com/users/sirreal/gists{/gist_id}","starred_url":"https://api.github.com/users/sirreal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sirreal/subscriptions","organizations_url":"https://api.github.com/users/sirreal/orgs","repos_url":"https://api.github.com/users/sirreal/repos","events_url":"https://api.github.com/users/sirreal/events{/privacy}","received_events_url":"https://api.github.com/users/sirreal/received_events","type":"User","site_admin":false}],"milestone":null,"comments":10,"created_at":"2018-03-22T17:20:30Z","updated_at":"2018-03-23T14:14:19Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/23554","html_url":"https://github.com/Automattic/wp-calypso/pull/23554","diff_url":"https://github.com/Automattic/wp-calypso/pull/23554.diff","patch_url":"https://github.com/Automattic/wp-calypso/pull/23554.patch"},"body":"We were loading huge images in the theme view. Reduce the size by using Photon.\r\nAlso adds `srcset` with 2x images.\r\n\r\nThis should be viable for `wporg` themes as well, but the the screenshot urls that appear fine in state are having `?ver=…` appended to them, it's unclear why. In this case the non-photon url is used as a fallback..\r\n\r\nRelated: #19893 #20009\r\n\r\n## Screens\r\n\r\n## Before\r\n\r\nNote the mismatch native/actual sizes:\r\n\r\n![before](https://user-images.githubusercontent.com/841763/37786771-bd5daf0c-2dfd-11e8-8fde-2fd3f1aee8d5.png)\r\n\r\nNote the bad query (`?w=2880?w=680`)\r\n![before-html](https://user-images.githubusercontent.com/841763/37786772-bd8749c0-2dfd-11e8-9b92-04f5361da7c5.png)\r\n\r\n## After\r\n![after-html](https://user-images.githubusercontent.com/841763/37786768-bd0896d4-2dfd-11e8-9f56-0e0394ece19c.png)\r\n![after](https://user-images.githubusercontent.com/841763/37786769-bd32aa78-2dfd-11e8-8f02-cc46da445710.png)\r\n\r\n## Testing\r\n1. Test logged-in and logged-out\r\n1. Test WordPress.com and Jetpack sites\r\n1. On a single theme page (`/theme/THEME_SLUG/MAYBE_JETPACK_SITE_SLUG`), verify everything works and displays correctly and there are no errors or warnings on the console.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/Automattic/wp-calypso/issues/comments/375678436","html_url":"https://github.com/Automattic/wp-calypso/pull/23554#issuecomment-375678436","issue_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/23554","id":375678436,"user":{"login":"ockham","id":96308,"avatar_url":"https://avatars2.githubusercontent.com/u/96308?v=4","gravatar_id":"","url":"https://api.github.com/users/ockham","html_url":"https://github.com/ockham","followers_url":"https://api.github.com/users/ockham/followers","following_url":"https://api.github.com/users/ockham/following{/other_user}","gists_url":"https://api.github.com/users/ockham/gists{/gist_id}","starred_url":"https://api.github.com/users/ockham/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ockham/subscriptions","organizations_url":"https://api.github.com/users/ockham/orgs","repos_url":"https://api.github.com/users/ockham/repos","events_url":"https://api.github.com/users/ockham/events{/privacy}","received_events_url":"https://api.github.com/users/ockham/received_events","type":"User","site_admin":false},"created_at":"2018-03-23T14:14:19Z","updated_at":"2018-03-23T14:14:19Z","author_association":"CONTRIBUTOR","body":"(As in, have photon pass it through to the original image location?)"}}
{"actor":{"display_login":"ockham"}}