Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[bug] Row info about visibility is missing #2486

Open
ybelenko opened this issue Jan 3, 2022 · 3 comments
Open

[bug] Row info about visibility is missing #2486

ybelenko opened this issue Jan 3, 2022 · 3 comments
Labels
good first issue

Comments

@ybelenko
Copy link

@ybelenko ybelenko commented Jan 3, 2022 •
edited

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".
@SheetJSDev
Copy link
Contributor

@SheetJSDev SheetJSDev commented Jan 5, 2022

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.

@SheetJSDev SheetJSDev closed this Jan 5, 2022
@ybelenko
Copy link
Author

@ybelenko ybelenko commented Jan 15, 2022

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
Copy link
Contributor

@reviewher reviewher commented Jan 15, 2022

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

@reviewher reviewher reopened this Jan 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@ybelenko @SheetJSDev @reviewher