Skip to content

Commit 9ac8c4a

Browse files
dazgreerCopilot
andcommitted
fix(mjml-section / skeleton): column-align default updated / removed apple message formatting
- Updated column-align default and inheritance from text-align as the colum-align default was overriding the user specified text-align value - removed <meta name="x-apple-disable-message-reformatting"> from skeleton as it's legacy code (iOS9 and bellow) Co-authored-by: Copilot <copilot@github.com>
1 parent ba1c9af commit 9ac8c4a

2 files changed

Lines changed: 1 addition & 3 deletions

File tree

packages/mjml-core/src/helpers/skeleton.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ export default function skeleton(options) {
3636
<meta charset="utf-8">
3737
<meta name="viewport" content="width=device-width, initial-scale=1, user-scalable=yes">
3838
<meta name="format-detection" content="telephone=no, date=no, address=no, email=no, url=no">
39-
<meta name="x-apple-disable-message-reformatting">
4039
${
4140
supportDarkMode
4241
? '<meta name="color-scheme" content="light dark">\n <meta name="supported-color-schemes" content="light dark">'

packages/mjml-section/src/index.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,13 @@ export default class MjSection extends BodyComponent {
3535

3636
static defaultAttributes = {
3737
'background-position': 'top center',
38-
'column-align': 'center',
3938
padding: '20px 0',
4039
'text-align': 'center',
4140
'text-padding': '4px 4px 4px 0',
4241
}
4342

4443
getColumnAlign() {
45-
return this.getAttribute('column-align') || this.getAttribute('text-align')
44+
return this.getAttribute('column-align') || this.getAttribute('text-align') || 'center'
4645
}
4746

4847
getChildContext() {

0 commit comments

Comments
 (0)