fix build on travis + pxpx in background for outlook

mjmlio · Nov 28, 2016 · f0e49a6 · f0e49a6
1 parent 16d1cf7
commit f0e49a6
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -49,7 +49,7 @@
     "vars-on-top": 0,
 
     "import/extensions": 0,
-    "import/imports-first": 0,
+    "import/first": 0,
     "import/no-dynamic-require": 0,
     "import/no-extraneous-dependencies": 0,
     "import/no-unresolved": 0,

diff --git a/package.json b/package.json
@@ -23,7 +23,7 @@
     "babel-preset-stage-0": "^6.16.0",
     "babel-register": "^6.16.3",
     "eslint": "^3.8.1",
-    "eslint-config-airbnb": "^12.0.0",
+    "eslint-config-airbnb": "^13.0.0",
     "eslint-plugin-import": "^2.0.1",
     "eslint-plugin-jsx-a11y": "^2.2.3",
     "eslint-plugin-react": "^6.4.1",

diff --git a/packages/mjml-section/src/index.js b/packages/mjml-section/src/index.js
@@ -43,7 +43,7 @@ const baseStyles = {
 const postRender = $ => {
   $('.mj-section-outlook-background').each(function () {
     const url = $(this).data('url')
-    const width = $(this).data('width')
+    const width = parseInt($(this).data('width'))
 
     $(this)
       .removeAttr('class')
@@ -78,9 +78,10 @@ const postRender = $ => {
 
   $('.mj-section-outlook-line').each(function () {
     const $columnDiv = $(this).next()
+    const width = parseInt($(this).data('width'))
 
     $(this).replaceWith(`${helpers.startConditionalTag}
-      </td><td style="vertical-align:${$columnDiv.data('vertical-align')};width:${parseInt($(this).data('width'))}px;">
+      </td><td style="vertical-align:${$columnDiv.data('vertical-align')};width:${width}px;">
       ${helpers.endConditionalTag}`)
 
     $columnDiv.removeAttr('data-vertical-align')