exceljs
diff --git a/‎lib/xlsx/xform/sheet/col-xform.js
Lines changed: 1 addition & 1 deletion b/‎lib/xlsx/xform/sheet/col-xform.js
Lines changed: 1 addition & 1 deletion
diff --git a/‎lib/xlsx/xform/sheet/row-xform.js
Lines changed: 1 addition & 1 deletion b/‎lib/xlsx/xform/sheet/row-xform.js
Lines changed: 1 addition & 1 deletion
diff --git a/‎spec/integration/data/hidden-test/google-sheets.xlsx
4.66 KB b/‎spec/integration/data/hidden-test/google-sheets.xlsx
4.66 KB
diff --git a/‎spec/integration/data/hidden-test/libre-calc-as-excel-2007-365.xlsx
4.72 KB b/‎spec/integration/data/hidden-test/libre-calc-as-excel-2007-365.xlsx
4.72 KB
diff --git a/‎spec/integration/data/hidden-test/libre-calc-as-office-open-xml-spreadsheet.xlsx
4.73 KB b/‎spec/integration/data/hidden-test/libre-calc-as-office-open-xml-spreadsheet.xlsx
4.73 KB
diff --git a/‎spec/integration/worksheet.spec.js
Lines changed: 41 additions & 1 deletion b/‎spec/integration/worksheet.spec.js
Lines changed: 41 additions & 1 deletion
@@ -48,7 +48,7 @@ class ColXform extends BaseXform {
       if (node.attributes.style) {
         model.styleId = parseInt(node.attributes.style, 10);
       }
-      if (node.attributes.hidden) {
+      if (node.attributes.hidden === true || node.attributes.hidden === 'true' || node.attributes.hidden === 1 || node.attributes.hidden === '1') {
         model.hidden = true;
       }
       if (node.attributes.bestFit) {
 
@@ -77,7 +77,7 @@ class RowXform extends BaseXform {
       if (node.attributes.s) {
         model.styleId = parseInt(node.attributes.s, 10);
       }
-      if (node.attributes.hidden) {
+      if (node.attributes.hidden === true || node.attributes.hidden === 'true' || node.attributes.hidden === 1 || node.attributes.hidden === '1') {
         model.hidden = true;
       }
       if (node.attributes.bestFit) {
 
@@ -1,6 +1,6 @@
 const path = require('path');
 
-const testutils = require('./../utils/index');
+const testutils = require('../utils/index');
 
 const ExcelJS = verquire('exceljs');
 const Range = verquire('doc/range');
@@ -812,4 +812,44 @@ describe('Worksheet', () => {
         expect(workbook).to.have.property('worksheets');
         expect(workbook.worksheets).to.have.length(1);
       }));
+
+  describe('Hidden', () => {
+    const fileList = [
+      'google-sheets',
+      'libre-calc-as-excel-2007-365',
+      'libre-calc-as-office-open-xml-spreadsheet',
+    ];
+
+    for (const file of fileList) {
+      it(`Should set hidden attribute correctly (${file})`, done => {
+        const wb = new ExcelJS.Workbook();
+        wb.xlsx
+          .readFile(
+            path.resolve(__dirname, 'data', 'hidden-test', `${file}.xlsx`)
+          )
+          .then(() => {
+            const ws = wb.getWorksheet(1);
+
+            //  Check rows
+            expect(ws.getRow(1).hidden, `${file} : Row 1`).to.equal(false);
+            expect(ws.getRow(2).hidden, `${file} : Row 2`).to.equal(true);
+            expect(ws.getRow(3).hidden, `${file} : Row 3`).to.equal(false);
+
+            //  Check columns
+            expect(ws.getColumn(1).hidden, `${file} : Column 1`).to.equal(
+              false
+            );
+            expect(ws.getColumn(2).hidden, `${file} : Column 2`).to.equal(true);
+            expect(ws.getColumn(3).hidden, `${file} : Column 3`).to.equal(
+              false
+            );
+
+            done();
+          })
+          .catch(error => {
+            done(error);
+          });
+      });
+    }
+  });
 });
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ class ColXform extends BaseXform {`
`48`	`48`	`if (node.attributes.style) {`
`49`	`49`	`model.styleId = parseInt(node.attributes.style, 10);`
`50`	`50`	`}`
`51`		`- if (node.attributes.hidden) {`
	`51`	`+ if (node.attributes.hidden === true \|\| node.attributes.hidden === 'true' \|\| node.attributes.hidden === 1 \|\| node.attributes.hidden === '1') {`
`52`	`52`	`model.hidden = true;`
`53`	`53`	`}`
`54`	`54`	`if (node.attributes.bestFit) {`
Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@ class RowXform extends BaseXform {`
`77`	`77`	`if (node.attributes.s) {`
`78`	`78`	`model.styleId = parseInt(node.attributes.s, 10);`
`79`	`79`	`}`
`80`		`- if (node.attributes.hidden) {`
	`80`	`+ if (node.attributes.hidden === true \|\| node.attributes.hidden === 'true' \|\| node.attributes.hidden === 1 \|\| node.attributes.hidden === '1') {`
`81`	`81`	`model.hidden = true;`
`82`	`82`	`}`
`83`	`83`	`if (node.attributes.bestFit) {`