{"action":"created","issue":{"url":"https://api.github.com/repos/Automattic/jetpack/issues/4312","repository_url":"https://api.github.com/repos/Automattic/jetpack","labels_url":"https://api.github.com/repos/Automattic/jetpack/issues/4312/labels{/name}","comments_url":"https://api.github.com/repos/Automattic/jetpack/issues/4312/comments","events_url":"https://api.github.com/repos/Automattic/jetpack/issues/4312/events","html_url":"https://github.com/Automattic/jetpack/issues/4312","id":164068237,"number":4312,"title":"instagram shortcode example: width=300 less than minimum 320","user":{"login":"bobbingwide","id":2474435,"avatar_url":"https://avatars.githubusercontent.com/u/2474435?v=3","gravatar_id":"","url":"https://api.github.com/users/bobbingwide","html_url":"https://github.com/bobbingwide","followers_url":"https://api.github.com/users/bobbingwide/followers","following_url":"https://api.github.com/users/bobbingwide/following{/other_user}","gists_url":"https://api.github.com/users/bobbingwide/gists{/gist_id}","starred_url":"https://api.github.com/users/bobbingwide/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bobbingwide/subscriptions","organizations_url":"https://api.github.com/users/bobbingwide/orgs","repos_url":"https://api.github.com/users/bobbingwide/repos","events_url":"https://api.github.com/users/bobbingwide/events{/privacy}","received_events_url":"https://api.github.com/users/bobbingwide/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-07-06T12:52:01Z","updated_at":"2016-07-06T13:46:15Z","closed_at":null,"body":"<!-- Thanks for contributing to Jetpack! Pick a clear title (\"Sharing: add new Facebook button\") and proceed. -->\r\n\r\n#### Steps to reproduce the issue\r\nCreate a post with two instagram shortcodes copied from the examples in  modules/shortcodes/instagram.php \r\n\r\n```\r\n// [instagram url=\"http://instagram.com/p/PSbF9sEIGP/\"]\r\n// [instagram url=\"http://instagram.com/p/PSbF9sEIGP/\" width=\"300\"]\r\n\r\n```\r\n#### What I expected\r\nHaving not read the code properly I expected the second example to be narrower than the first.\r\n\r\n#### What happened instead\r\nIt was wider. \r\n\r\nI then checked the help on WordPress.com, and then the code, and found that the minimum width supported by the shortcode is 320. I didn't check Instagram's limits.\r\n\r\n#### Suggestion\r\nChange the second example to `width=\"320\"`\r\n\r\n<!--\r\nPLEASE NOTE\r\n- These comments won't show up when you submit the issue.\r\n- Everything is optional, but try to add as many details as possible.\r\n- If requesting a new feature, explain why you'd like to see it added.\r\n- This issue tracker is not for support. If you have questions about Jetpack, you can [start a new thread in the Jetpack support forums](http://wordpress.org/support/plugin/jetpack#postform), or [send us an email](http://jetpack.com/contact-support/).\r\n- Do not report potential security vulnerabilities here. For responsible disclosure of security issues and to be eligible for our bug bounty program, please submit your report via [the HackerOne portal](https://hackerone.com/automattic).\r\n-->\r\n"},"comment":{"url":"https://api.github.com/repos/Automattic/jetpack/issues/comments/230775745","html_url":"https://github.com/Automattic/jetpack/issues/4312#issuecomment-230775745","issue_url":"https://api.github.com/repos/Automattic/jetpack/issues/4312","id":230775745,"user":{"login":"bobbingwide","id":2474435,"avatar_url":"https://avatars.githubusercontent.com/u/2474435?v=3","gravatar_id":"","url":"https://api.github.com/users/bobbingwide","html_url":"https://github.com/bobbingwide","followers_url":"https://api.github.com/users/bobbingwide/followers","following_url":"https://api.github.com/users/bobbingwide/following{/other_user}","gists_url":"https://api.github.com/users/bobbingwide/gists{/gist_id}","starred_url":"https://api.github.com/users/bobbingwide/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bobbingwide/subscriptions","organizations_url":"https://api.github.com/users/bobbingwide/orgs","repos_url":"https://api.github.com/users/bobbingwide/repos","events_url":"https://api.github.com/users/bobbingwide/events{/privacy}","received_events_url":"https://api.github.com/users/bobbingwide/received_events","type":"User","site_admin":false},"created_at":"2016-07-06T13:46:15Z","updated_at":"2016-07-06T13:46:15Z","body":"Instagram's minimum is 320. https://www.instagram.com/developer/embedding/\r\n\r\nIs it just me or does anyone else think that the current logic setting the width to max_width is rather odd. Too high a max_width gets reduced to the maximum. I'd expect too low a min_width to be increased to the minimum, not the maximum. "}}
{"actor":{"display_login":"bobbingwide"}}