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

Paste cells command behavior option by Contrail Co.,Ltd. #4777

Merged

Conversation

shun-iwasawa
Copy link
Member

@shun-iwasawa shun-iwasawa commented Mar 13, 2023

In most cases in Japanese animation production, one xsheet column corresponds to one level. Therefore, there are few cases where the entire contents of a cell (i.e. level and drawing number) in one column are pasted into another column, and pasting of drawing numbers is more frequently required.

Therefore, this PR will add a preference item Paste Cells Behaviour to Xsheet category, with the options as follows;

  • Insert Paste Whole Data (default option) : The command will paste entire contents of copied cells.
  • Overwrite Paste Cell Numbers : The command will execute Paste Cell Numbers instead.

When the latter option is selected, a command Paste Whole Cell Data Paste Cell Content will appear in the right click menu of xsheet, in Paste Special submenu. This will allow users to use the conventional paste behavior.

This improvement was developed on consignment and is copyrighted by Contrail Co., Ltd.

@gab3d
Copy link
Contributor

gab3d commented Mar 13, 2023

Would Paste Cell Content be an equivalent but shorter alternative to the Paste Whole Cell Data command name?

Also, what's the effect of this new preference option over assigned keyboard shortcuts?

@shun-iwasawa
Copy link
Member Author

@gab3d

what's the effect of this new preference option over assigned keyboard shortcuts?

With the latter option , using the shortcut key for the command Paste Insert (Ctrl+V by default) will act as Paste Cell Numbers. Assigning the frequent paste behavior to Ctrl+V is actually the main purpose of this modification.

@RodneyBaker
Copy link
Collaborator

Thanks again to Contrail Co. ltd for the new options.
LGTM

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.

None yet

3 participants