Skip to content

Commit

Permalink
removes tbody tag from wrapper causing issues in outlook
Browse files Browse the repository at this point in the history
  • Loading branch information
RafiBomb committed Mar 31, 2016
1 parent b83851e commit ea1888b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
2 changes: 1 addition & 1 deletion lib/componentFactory.js
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ module.exports = function(element) {
classes = classes.concat(element.attr('class').split(' '));
}

return format('<table class="%s"><tbody><tr><td class="wrapper-inner">%s</td></tr></tbody></table>', classes.join(' '), inner);
return format('<table class="%s"><tr><td class="wrapper-inner">%s</td></tr></table>', classes.join(' '), inner);

default:
// If it's not a custom component, return it as-is
Expand Down
8 changes: 3 additions & 5 deletions test/components.js
Original file line number Diff line number Diff line change
Expand Up @@ -267,11 +267,9 @@ describe('wrapper', () => {
var input = `<wrapper class="header"></wrapper>`;
var expected = `
<table class="wrapper header">
<tbody>
<tr>
<td class="wrapper-inner"></td>
</tr>
</tbody>
<tr>
<td class="wrapper-inner"></td>
</tr>
</table>
`;

Expand Down

0 comments on commit ea1888b

Please sign in to comment.