{"issue":{"created_at":"2011-11-15T23:59:33Z","state":"open","updated_at":"2011-12-01T20:48:44Z","comments":1,"user":{"gravatar_id":"3812246319a3e4c7c292dbe3f13c7efa","id":568164,"url":"https://api.github.com/users/andyadams","login":"andyadams","avatar_url":"https://secure.gravatar.com/avatar/3812246319a3e4c7c292dbe3f13c7efa?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"References issue #10. When the maxWidth option is set to true, an extra container is added that limits the size of the video to the originally-specified dimensions.\r\n\r\nMy text editor also strips out trailing whitespace, so there's a bunch of that removed as well :).\r\n\r\nP.S. Chris, at WordCamp SF I totally ditched mid-talk upstairs to come and hear yours. It was awesome.","assignee":null,"id":2251115,"labels":[],"pull_request":{"patch_url":"https://github.com/davatron5000/FitVids.js/pull/19.patch","html_url":"https://github.com/davatron5000/FitVids.js/pull/19","diff_url":"https://github.com/davatron5000/FitVids.js/pull/19.diff"},"number":19,"title":"Adding maxWidth variable that adds wrapper, retains original max width/height","html_url":"https://github.com/davatron5000/FitVids.js/issues/19","closed_at":null,"url":"https://api.github.com/repos/davatron5000/FitVids.js/issues/19","milestone":null},"action":"created","comment":{"created_at":"2011-12-01T20:48:44Z","updated_at":"2011-12-01T20:48:44Z","user":{"gravatar_id":"c34a1644cd28f2e410a04e5ca675ece9","id":1154110,"url":"https://api.github.com/users/joesavage","login":"joesavage","avatar_url":"https://secure.gravatar.com/avatar/c34a1644cd28f2e410a04e5ca675ece9?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"The maxWidth functionality is really nice, but in the cases where I've tried to use it - it has proved something I've just had to put back to false. This is because stopping the video being 100% width, means that the video isn't centered within it's parent division (and hence looks completely ugly on the page).\r\nI can't really think of any obvious ways to fix this though. Would I be right in saying that it would require some heavier Javascript to center within the parent with a max-width (or am I just being stupid)?","id":2980368,"url":"https://api.github.com/repos/davatron5000/FitVids.js/issues/comments/2980368"}}