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

Add method to hide rows/columns #1012

Merged
merged 1 commit into from
Mar 20, 2022
Merged

Add method to hide rows/columns #1012

merged 1 commit into from
Mar 20, 2022

Conversation

lavigne958
Copy link
Collaborator

Add new method that allows the user to explicitly hide rows or columns.
Ass new method that allows the user to explicitly unhide rows or
columns.

This feature does not include any listing method, because the listing
is not in the metadata of the SpreadSheet or the Sheet, it will
require to list all the data in the sheet and list all of them
then check if the flag hiddenByUser is set. This is a lot of
processing and with very large data set this can take some time
For a result that is only visible using the UI.

closes #1006

Add new method that allows the user to explicitly hide rows or columns.
Ass new method that allows the user to explicitly unhide rows or
columns.

This feature does not include any _listing_ method, because the listing
is not in the metadata of the `SpreadSheet` or the `Sheet`, it will
require to list all the data in the sheet and list all of them
then check if the flag `hiddenByUser` is set. This is a lot of
processing and with very large data set this can take some time
For a result that is only visible using the UI.

closes #1006
@lavigne958
Copy link
Collaborator Author

@juliusdanek what do you think ?

@juliusdanek
Copy link

Looks excellent! Thank you sir.

@lavigne958 lavigne958 merged commit f4c4780 into master Mar 20, 2022
@lavigne958 lavigne958 deleted the feature/hide_cols_rows branch March 28, 2022 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ability to hide columns and/or rows
2 participants