{"action":"created","comment":{"url":"https://api.github.com/repos/Automattic/juice/pulls/comments/38261756","id":38261756,"diff_hunk":"@@ -220,10 +220,16 @@ function inlineDocument($, css, options) {\n     var elName = el.name.toUpperCase();\n     if (juice.widthElements.indexOf(elName) > -1) {\n       for (var i in el.styleProps) {\n-        if (el.styleProps[i].prop === 'width' && el.styleProps[i].value.match(/px/)) {\n-          var pxWidth = el.styleProps[i].value.replace('px', '');\n-          $(el).attr('width', pxWidth);\n-          return;\n+        if (el.styleProps[i].prop === 'width') {\n+          if (el.styleProps[i].value.match(/px/)) {\n+            var pxWidth = el.styleProps[i].value.replace('px', '');\n+            $(el).attr('width', pxWidth);\n+            return;\n+          }\n+          if (juice.tableElements.indexOf(elName) > -1 && el.styleProps[i].value.match(/\\%/)) {","path":"lib/juice.js","position":14,"original_position":14,"commit_id":"0aa8e02a3ff6f5e6efac4f8ca4f4c9229fc1e121","original_commit_id":"b17ea02f1faaffc175f4f34f9657fd9a212f0519","user":{"login":"jrit","id":1255823,"avatar_url":"https://avatars.githubusercontent.com/u/1255823?v=3","gravatar_id":"","url":"https://api.github.com/users/jrit","html_url":"https://github.com/jrit","followers_url":"https://api.github.com/users/jrit/followers","following_url":"https://api.github.com/users/jrit/following{/other_user}","gists_url":"https://api.github.com/users/jrit/gists{/gist_id}","starred_url":"https://api.github.com/users/jrit/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jrit/subscriptions","organizations_url":"https://api.github.com/users/jrit/orgs","repos_url":"https://api.github.com/users/jrit/repos","events_url":"https://api.github.com/users/jrit/events{/privacy}","received_events_url":"https://api.github.com/users/jrit/received_events","type":"User","site_admin":false},"body":"Oh, yeah, I was thinking about setting image widths in `style` which is obviously not what this does. Do you mind just updating tests to cover this and then it can be merged?","created_at":"2015-08-29T12:52:30Z","updated_at":"2015-08-29T12:52:30Z","html_url":"https://github.com/Automattic/juice/pull/153#discussion_r38261756","pull_request_url":"https://api.github.com/repos/Automattic/juice/pulls/153","_links":{"self":{"href":"https://api.github.com/repos/Automattic/juice/pulls/comments/38261756"},"html":{"href":"https://github.com/Automattic/juice/pull/153#discussion_r38261756"},"pull_request":{"href":"https://api.github.com/repos/Automattic/juice/pulls/153"}}},"pull_request":{"url":"https://api.github.com/repos/Automattic/juice/pulls/153","id":43571853,"html_url":"https://github.com/Automattic/juice/pull/153","diff_url":"https://github.com/Automattic/juice/pull/153.diff","patch_url":"https://github.com/Automattic/juice/pull/153.patch","issue_url":"https://api.github.com/repos/Automattic/juice/issues/153","number":153,"state":"open","locked":false,"title":"Enabled percentages for tableElements on 'applyWidthAttributes' option.","user":{"login":"hectormenendez","id":154102,"avatar_url":"https://avatars.githubusercontent.com/u/154102?v=3","gravatar_id":"","url":"https://api.github.com/users/hectormenendez","html_url":"https://github.com/hectormenendez","followers_url":"https://api.github.com/users/hectormenendez/followers","following_url":"https://api.github.com/users/hectormenendez/following{/other_user}","gists_url":"https://api.github.com/users/hectormenendez/gists{/gist_id}","starred_url":"https://api.github.com/users/hectormenendez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hectormenendez/subscriptions","organizations_url":"https://api.github.com/users/hectormenendez/orgs","repos_url":"https://api.github.com/users/hectormenendez/repos","events_url":"https://api.github.com/users/hectormenendez/events{/privacy}","received_events_url":"https://api.github.com/users/hectormenendez/received_events","type":"User","site_admin":false},"body":"Mapping percentages width attributes in table elements is useful for fluid layouts and does not break any specs.","created_at":"2015-08-27T21:57:00Z","updated_at":"2015-08-29T12:52:30Z","closed_at":null,"merged_at":null,"merge_commit_sha":"174a47778a9ddfde146e959598243209c43ef76f","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/Automattic/juice/pulls/153/commits","review_comments_url":"https://api.github.com/repos/Automattic/juice/pulls/153/comments","review_comment_url":"https://api.github.com/repos/Automattic/juice/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Automattic/juice/issues/153/comments","statuses_url":"https://api.github.com/repos/Automattic/juice/statuses/0aa8e02a3ff6f5e6efac4f8ca4f4c9229fc1e121","head":{"label":"hectormenendez:master","ref":"master","sha":"0aa8e02a3ff6f5e6efac4f8ca4f4c9229fc1e121","user":{"login":"hectormenendez","id":154102,"avatar_url":"https://avatars.githubusercontent.com/u/154102?v=3","gravatar_id":"","url":"https://api.github.com/users/hectormenendez","html_url":"https://github.com/hectormenendez","followers_url":"https://api.github.com/users/hectormenendez/followers","following_url":"https://api.github.com/users/hectormenendez/following{/other_user}","gists_url":"https://api.github.com/users/hectormenendez/gists{/gist_id}","starred_url":"https://api.github.com/users/hectormenendez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hectormenendez/subscriptions","organizations_url":"https://api.github.com/users/hectormenendez/orgs","repos_url":"https://api.github.com/users/hectormenendez/repos","events_url":"https://api.github.com/users/hectormenendez/events{/privacy}","received_events_url":"https://api.github.com/users/hectormenendez/received_events","type":"User","site_admin":false},"repo":{"id":41512087,"name":"juice","full_name":"hectormenendez/juice","owner":{"login":"hectormenendez","id":154102,"avatar_url":"https://avatars.githubusercontent.com/u/154102?v=3","gravatar_id":"","url":"https://api.github.com/users/hectormenendez","html_url":"https://github.com/hectormenendez","followers_url":"https://api.github.com/users/hectormenendez/followers","following_url":"https://api.github.com/users/hectormenendez/following{/other_user}","gists_url":"https://api.github.com/users/hectormenendez/gists{/gist_id}","starred_url":"https://api.github.com/users/hectormenendez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hectormenendez/subscriptions","organizations_url":"https://api.github.com/users/hectormenendez/orgs","repos_url":"https://api.github.com/users/hectormenendez/repos","events_url":"https://api.github.com/users/hectormenendez/events{/privacy}","received_events_url":"https://api.github.com/users/hectormenendez/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/hectormenendez/juice","description":"Juice inlines CSS stylesheets into your HTML source.","fork":true,"url":"https://api.github.com/repos/hectormenendez/juice","forks_url":"https://api.github.com/repos/hectormenendez/juice/forks","keys_url":"https://api.github.com/repos/hectormenendez/juice/keys{/key_id}","collaborators_url":"https://api.github.com/repos/hectormenendez/juice/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/hectormenendez/juice/teams","hooks_url":"https://api.github.com/repos/hectormenendez/juice/hooks","issue_events_url":"https://api.github.com/repos/hectormenendez/juice/issues/events{/number}","events_url":"https://api.github.com/repos/hectormenendez/juice/events","assignees_url":"https://api.github.com/repos/hectormenendez/juice/assignees{/user}","branches_url":"https://api.github.com/repos/hectormenendez/juice/branches{/branch}","tags_url":"https://api.github.com/repos/hectormenendez/juice/tags","blobs_url":"https://api.github.com/repos/hectormenendez/juice/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/hectormenendez/juice/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/hectormenendez/juice/git/refs{/sha}","trees_url":"https://api.github.com/repos/hectormenendez/juice/git/trees{/sha}","statuses_url":"https://api.github.com/repos/hectormenendez/juice/statuses/{sha}","languages_url":"https://api.github.com/repos/hectormenendez/juice/languages","stargazers_url":"https://api.github.com/repos/hectormenendez/juice/stargazers","contributors_url":"https://api.github.com/repos/hectormenendez/juice/contributors","subscribers_url":"https://api.github.com/repos/hectormenendez/juice/subscribers","subscription_url":"https://api.github.com/repos/hectormenendez/juice/subscription","commits_url":"https://api.github.com/repos/hectormenendez/juice/commits{/sha}","git_commits_url":"https://api.github.com/repos/hectormenendez/juice/git/commits{/sha}","comments_url":"https://api.github.com/repos/hectormenendez/juice/comments{/number}","issue_comment_url":"https://api.github.com/repos/hectormenendez/juice/issues/comments{/number}","contents_url":"https://api.github.com/repos/hectormenendez/juice/contents/{+path}","compare_url":"https://api.github.com/repos/hectormenendez/juice/compare/{base}...{head}","merges_url":"https://api.github.com/repos/hectormenendez/juice/merges","archive_url":"https://api.github.com/repos/hectormenendez/juice/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/hectormenendez/juice/downloads","issues_url":"https://api.github.com/repos/hectormenendez/juice/issues{/number}","pulls_url":"https://api.github.com/repos/hectormenendez/juice/pulls{/number}","milestones_url":"https://api.github.com/repos/hectormenendez/juice/milestones{/number}","notifications_url":"https://api.github.com/repos/hectormenendez/juice/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/hectormenendez/juice/labels{/name}","releases_url":"https://api.github.com/repos/hectormenendez/juice/releases{/id}","created_at":"2015-08-27T21:23:24Z","updated_at":"2015-08-27T21:23:25Z","pushed_at":"2015-08-29T05:32:26Z","git_url":"git://github.com/hectormenendez/juice.git","ssh_url":"git@github.com:hectormenendez/juice.git","clone_url":"https://github.com/hectormenendez/juice.git","svn_url":"https://github.com/hectormenendez/juice","homepage":"","size":942,"stargazers_count":0,"watchers_count":0,"language":"JavaScript","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"Automattic:master","ref":"master","sha":"7f0757ae27d7c87026c0f42acb89cfce8af16ac5","user":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"repo":{"id":2536587,"name":"juice","full_name":"Automattic/juice","owner":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Automattic/juice","description":"Juice inlines CSS stylesheets into your HTML source.","fork":false,"url":"https://api.github.com/repos/Automattic/juice","forks_url":"https://api.github.com/repos/Automattic/juice/forks","keys_url":"https://api.github.com/repos/Automattic/juice/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Automattic/juice/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Automattic/juice/teams","hooks_url":"https://api.github.com/repos/Automattic/juice/hooks","issue_events_url":"https://api.github.com/repos/Automattic/juice/issues/events{/number}","events_url":"https://api.github.com/repos/Automattic/juice/events","assignees_url":"https://api.github.com/repos/Automattic/juice/assignees{/user}","branches_url":"https://api.github.com/repos/Automattic/juice/branches{/branch}","tags_url":"https://api.github.com/repos/Automattic/juice/tags","blobs_url":"https://api.github.com/repos/Automattic/juice/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Automattic/juice/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Automattic/juice/git/refs{/sha}","trees_url":"https://api.github.com/repos/Automattic/juice/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Automattic/juice/statuses/{sha}","languages_url":"https://api.github.com/repos/Automattic/juice/languages","stargazers_url":"https://api.github.com/repos/Automattic/juice/stargazers","contributors_url":"https://api.github.com/repos/Automattic/juice/contributors","subscribers_url":"https://api.github.com/repos/Automattic/juice/subscribers","subscription_url":"https://api.github.com/repos/Automattic/juice/subscription","commits_url":"https://api.github.com/repos/Automattic/juice/commits{/sha}","git_commits_url":"https://api.github.com/repos/Automattic/juice/git/commits{/sha}","comments_url":"https://api.github.com/repos/Automattic/juice/comments{/number}","issue_comment_url":"https://api.github.com/repos/Automattic/juice/issues/comments{/number}","contents_url":"https://api.github.com/repos/Automattic/juice/contents/{+path}","compare_url":"https://api.github.com/repos/Automattic/juice/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Automattic/juice/merges","archive_url":"https://api.github.com/repos/Automattic/juice/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Automattic/juice/downloads","issues_url":"https://api.github.com/repos/Automattic/juice/issues{/number}","pulls_url":"https://api.github.com/repos/Automattic/juice/pulls{/number}","milestones_url":"https://api.github.com/repos/Automattic/juice/milestones{/number}","notifications_url":"https://api.github.com/repos/Automattic/juice/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Automattic/juice/labels{/name}","releases_url":"https://api.github.com/repos/Automattic/juice/releases{/id}","created_at":"2011-10-08T03:14:11Z","updated_at":"2015-08-29T10:22:08Z","pushed_at":"2015-08-29T05:32:27Z","git_url":"git://github.com/Automattic/juice.git","ssh_url":"git@github.com:Automattic/juice.git","clone_url":"https://github.com/Automattic/juice.git","svn_url":"https://github.com/Automattic/juice","homepage":"","size":942,"stargazers_count":889,"watchers_count":889,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":105,"mirror_url":null,"open_issues_count":13,"forks":105,"open_issues":13,"watchers":889,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Automattic/juice/pulls/153"},"html":{"href":"https://github.com/Automattic/juice/pull/153"},"issue":{"href":"https://api.github.com/repos/Automattic/juice/issues/153"},"comments":{"href":"https://api.github.com/repos/Automattic/juice/issues/153/comments"},"review_comments":{"href":"https://api.github.com/repos/Automattic/juice/pulls/153/comments"},"review_comment":{"href":"https://api.github.com/repos/Automattic/juice/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Automattic/juice/pulls/153/commits"},"statuses":{"href":"https://api.github.com/repos/Automattic/juice/statuses/0aa8e02a3ff6f5e6efac4f8ca4f4c9229fc1e121"}}}}