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

[Question] Command Index counts weird after pasting #1036

Open
TheEmbracedOne opened this issue Oct 22, 2021 · 2 comments
Open

[Question] Command Index counts weird after pasting #1036

TheEmbracedOne opened this issue Oct 22, 2021 · 2 comments

Comments

@TheEmbracedOne
Copy link

If I move flowcharts from one scene to the other or copypaste commands or whatever, I notice that the number on the command doesnt start over. ie if i move a flowchart to a new scene, the first command on it doesnt become number (1), it retains its command count.

Does this ever cause a problem? Would it be a good idea to write a little extension to the flowchart script so that I could refresh these numbers with the click of a button?
image

@Arylos07
Copy link
Contributor

Arylos07 commented Oct 24, 2021

This isn't intended, but I don't think this will cause a problem. Those id's primarily serve to identify each component (block, command, etc) That would be problematic, but the flowchart also checks those id's on enable. Check Flowchart.OnEnable(). You'll see Flowchart.CheckItemIds(). That check looks through each command and makes sure a) the id's are all unique. b) they're in incrementing order. Since the flowchart does this when it's enabled, before any executions are performed, I think it will be a cosmetic problem at most since it's mostly used by editor functions rather than runtime functions.

@TheEmbracedOne
Copy link
Author

We've since talked about this on the Discord server and I just wanted to update this issue by saying that now I understand what these are for along with itemIDs, and that I decided to hide that label instead, since seeing this number is not useful to me, by commenting out the following in the CommandEditor.cs: GUILayout.Label(new GUIContent("(" + t.ItemId + ")"));
image

I'll leave this issue open just in case someone else has some input on it, or if it causes some problem later.

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

No branches or pull requests

2 participants