Fix conditional style fills

resolves issue 147: #147
natergj · Nov 25, 2017 · e081a88 · e081a88
1 parent 3a52197
commit e081a88
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -570,10 +570,10 @@ Style Options Object:
         outline: boolean
     },
     fill: { // §18.8.20 fill (Fill)
-        type: string, // Currently only 'pattern' is implimented. Non-implimented option is 'gradient'
+        type: string, // Currently only 'pattern' is implemented. Non-implemented option is 'gradient'
         patternType: string, //§18.18.55 ST_PatternType (Pattern Type)
-        bgColor: string // HTML style hex value. optional. defaults to black
-        fgColor: string // HTML style hex value. required.
+        bgColor: string // HTML style hex value. defaults to black
+        fgColor: string // HTML style hex value. defaults to black.
     },
     numberFormat: integer or string // §18.8.30 numFmt (Number Format)
 });

diff --git a/source/lib/style/style.js b/source/lib/style/style.js
@@ -320,7 +320,7 @@ class Style {
         }
 
         if (this.fill instanceof Fill) {
-            this.fill.addToXMLele(thisEle);
+            this.fill.addToXMLele(thisEle.ele('fill'));
         }
 
         if (this.alignment instanceof Alignment) {