[bug] Row info about visibility is missing #2486

New Issue

2022-01-03T23:36:29Z

ybelenko commented

2022-01-03 23:36:29 +00:00

(Migrated from github.com)

I'm trying to migrate from PHPOffice/PhpSpreadsheet to this package. PhpSpreadsheet works fine with my xlsx file, while sheetjs gives no info about hidden rows. It's really important since my script needs to ignore hidden cells.

Mentioned file has been sent to email from the Contributing doc.

Package version: "xlsx": "^0.17.4".

I'm trying to migrate from [PHPOffice/PhpSpreadsheet](https://github.com/PHPOffice/PhpSpreadsheet) to this package. PhpSpreadsheet works fine with my xlsx file, while sheetjs gives no info about hidden rows. It's really important since my script needs to ignore hidden cells. Mentioned file has been sent to email from the Contributing doc. Package version: `"xlsx": "^0.17.4".`

SheetJSDev commented

2022-01-05 20:38:45 +00:00

(Migrated from github.com)

Pass the option cellStyles: true to read, then the !rows property of the worksheet object will be populated. Each object corresponds to a row. The hidden property of each row will be set to true if hidden.

Pass the option `cellStyles: true` to read, then the `!rows` property of the worksheet object will be populated. Each object corresponds to a row. The `hidden` property of each row will be set to `true` if hidden.

👍 1 ❤️ 1

ybelenko commented

2022-01-15 15:00:38 +00:00

(Migrated from github.com)

Thanks, @SheetJSDev, it solves my problem.

It's not so obvious that row/column visibility stored in styles props and that it switched off by default. You should mention it in the main documentation. I was searching for an answer over there, but didn't find anything, so I've submitted an issue.

Thanks, @SheetJSDev, it solves my problem. It's not so obvious that row/column visibility stored in styles props and that it switched off by default. You should mention it in the main documentation. I was searching for an answer over there, but didn't find anything, so I've submitted an issue.

reviewher commented

2022-01-15 17:02:49 +00:00

(Migrated from github.com)

Feel free to submit a PR and change docbits/54_shobject.md

Feel free to submit a PR and change [`docbits/54_shobject.md`](https://github.com/SheetJS/sheetjs/blob/master/docbits/54_shobject.md)

ybelenko commented

2022-01-18 09:04:50 +00:00

(Migrated from github.com)

Oh, I'm flattered. Seems like you didn't notice that I'm not native english speaker. I'm not the right person to write official docs if you have someone from USA/UK.

azeez1776 commented

2022-01-26 14:40:40 +00:00

(Migrated from github.com)

@reviewher @ybelenko Can I work on the documentation issue and submit a PR

👍 1

azeez1776 commented

2022-01-27 12:41:08 +00:00

(Migrated from github.com)

@reviewher @ybelenko I submitted a PR

Sign in to join this conversation.