{"action":"opened","issue":{"url":"https://api.github.com/repos/servo/servo/issues/12531","repository_url":"https://api.github.com/repos/servo/servo","labels_url":"https://api.github.com/repos/servo/servo/issues/12531/labels{/name}","comments_url":"https://api.github.com/repos/servo/servo/issues/12531/comments","events_url":"https://api.github.com/repos/servo/servo/issues/12531/events","html_url":"https://github.com/servo/servo/issues/12531","id":166706642,"number":12531,"title":"Table cell vertical alignment impacted by line-height","user":{"login":"adamncasey","id":720498,"avatar_url":"https://avatars.githubusercontent.com/u/720498?v=3","gravatar_id":"","url":"https://api.github.com/users/adamncasey","html_url":"https://github.com/adamncasey","followers_url":"https://api.github.com/users/adamncasey/followers","following_url":"https://api.github.com/users/adamncasey/following{/other_user}","gists_url":"https://api.github.com/users/adamncasey/gists{/gist_id}","starred_url":"https://api.github.com/users/adamncasey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adamncasey/subscriptions","organizations_url":"https://api.github.com/users/adamncasey/orgs","repos_url":"https://api.github.com/users/adamncasey/repos","events_url":"https://api.github.com/users/adamncasey/events{/privacy}","received_events_url":"https://api.github.com/users/adamncasey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-07-20T23:37:08Z","updated_at":"2016-07-20T23:37:08Z","closed_at":null,"body":"```\r\n<html>\r\n<head>\r\n</head>\r\n<body>\r\n\t<table>\r\n\t\t\t<tr>\r\n\t\t\t\t<td style=\"line-height:32pt;\">Should be</td>\r\n\t\t\t\t<td>Level</td>\r\n\t\t\t</tr>\r\n\t</table>\r\n</body>\r\n</html>\r\n```\r\n\r\nIn Firefox, Chrome & Edge the text in these two cells is vertically aligned. \r\n\r\nIn Servo, the text in the first cell (with larger line-height applied) appears lower on the page than the text in the second cell.\r\n\r\nI had a look around `layout/table*.rs` but failed to make much headway. In https://github.com/servo/servo/blob/2d01d41a506bcbc7f26a2284b9f42390d6ef96ab/components/layout/table_row.rs#L108 the child cells of the row are assigned the correct heights (comparing against other browsers). Even so, the text in the cell without a line-height style still appears as if the entire row is the height of the smaller cell."}}
{"actor":{"display_login":"adamncasey"}}