Skip to content

Commit

Permalink
Apply badge border classes to child td instead of table (bootstrap-em…
Browse files Browse the repository at this point in the history
…ail#254)

This is necessary to make border-radius work.
  • Loading branch information
maurojs10 committed Jan 24, 2024
1 parent aedb535 commit 2c03264
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion core/templates/table-left.html
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<table class="{{ classes }}" align="left" role="presentation">
<tbody>
<tr>
<td>
<td class="{{ child_classes }}">
{{ contents }}
</td>
</tr>
Expand Down
5 changes: 4 additions & 1 deletion lib/bootstrap-email/converters/badge.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,10 @@ module Converter
class Badge < Base
def build
each_node('.badge') do |node|
node.replace(template('table-left', classes: node['class'], contents: node.delete('class') && node.to_html))
classes = node['class'].split(' ')
node_classes = classes.reject { |c| c =~ /^border/ }.join(' ')
child_classes = classes.select { |c| c =~ /^border/ }.join(' ')
node.replace(template('table-left', classes: node_classes, child_classes: child_classes, contents: node.delete('class') && node.to_html))
end
end
end
Expand Down

0 comments on commit 2c03264

Please sign in to comment.