Skip to content

Commit ce938b4

Browse files
committed
[FIX] mass_mailing: properly set new background colors in columns
Colorpicker snippets target <td> elements while these sometimes contain a <div> which is background-colored in css. In consequence, when using the colorpicker to change the background color of the snippet, the colored <div> came in front of the element of which we tried to change the color. This ensures the snippets target the <div> instead when the <td> itself is marked as non-colorable via the "o_mail_no_colorpicker" class, as this is then in fact the element we want to color. Enterprise themes need to be adapted to this new behavior.
1 parent b386eed commit ce938b4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

addons/mass_mailing/views/snippets_themes.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -576,7 +576,7 @@
576576
</td>
577577
</tr>
578578
<tr>
579-
<td>
579+
<td class="o_mail_no_colorpicker">
580580
<div class="bg-o-color-2">
581581
&amp;nbsp;
582582
</div>
@@ -865,7 +865,7 @@
865865
data-selector=".note-editable > div:not(.o_layout), .note-editable .oe_structure > div, td, th"
866866
data-exclude=".o_mail_no_resize, .o_mail_no_options"/>
867867

868-
<div data-selector=".note-editable > div:not(.o_layout), .note-editable .oe_structure > div, td, th"
868+
<div data-selector=".note-editable > div:not(.o_layout), .note-editable .oe_structure > div, td, td.o_mail_no_colorpicker div:first-child, th"
869869
data-exclude=".o_mail_no_colorpicker, .o_mail_no_options">
870870
<we-colorpicker string="Background Color"
871871
data-select-style="true"

0 commit comments

Comments
 (0)