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

Do not copy folded grids to clipboard #591

Closed
wants to merge 1 commit into from

Conversation

tobiolo
Copy link
Collaborator

@tobiolo tobiolo commented Jan 10, 2024

No description provided.

@aardappel
Copy link
Owner

So now "copy" does not copy folded cells but "export" does?
That might be confusing to the user.
What is the reason behind not wanting to copy folded cells?

@tobiolo
Copy link
Collaborator Author

tobiolo commented Jan 10, 2024

The main reason for me is what the user may expect in my eyes: If the cell has a folded grid, the user does not see the folded grid in the UI. So when the users copies to the clipboard, it should be consistent with what is displayed. If the user also wants to copy the folded grid, the user may unfold it; so it is copied to the clipboard what the user "sees". Do you agree? With export it is maybe more the expectation that it should include everything. The use case is especially for the situation where large stuff of content is folded and it should not be copied all to the clipboard because the user just wanted to copy at the higher (hierarchy) level and the stuff is intentially hidden away with the fold.

@aardappel
Copy link
Owner

I'm not sure if I agree that is what the user would expect. A user could expect either one, and not getting the folded cells when you expect them seems more problematic than getting the folded cells when you don't expect them. And then having no way to override that behavior.

If you get folded cells you didn't want you can delete them, if you are missing your folded cells you wanted you have to do the action again. Or if you don't notice, you now have a bad copy.

Did anyone complain about this behavior?

We could have a way to choose this behavior, but that is unnecessary complex UI.

Seems what we had before was on average better.

@tobiolo tobiolo closed this Jan 11, 2024
@tobiolo
Copy link
Collaborator Author

tobiolo commented Jan 11, 2024

I'm not sure if I agree that is what the user would expect. A user could expect either one, and not getting the folded cells when you expect them seems more problematic than getting the folded cells when you don't expect them. And then having no way to override that behavior.

If you get folded cells you didn't want you can delete them, if you are missing your folded cells you wanted you have to do the action again. Or if you don't notice, you now have a bad copy.

Did anyone complain about this behavior?

We could have a way to choose this behavior, but that is unnecessary complex UI.

Seems what we had before was on average better.

Ok, thanks for your reply.
I can understand the reasoning and I will close this PR then.

@tobiolo tobiolo deleted the folded-grids-no-copy branch March 13, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants