Notes for Node exports #10396
Open
Notes for Node exports #10396
+7
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tetrapod00
added
enhancement
topic:gdscript
area:manual
mhilbrunner
reviewed
Dec 15, 2024
mhilbrunner
reviewed
Dec 15, 2024
Co-authored-by: Max Hilbrunner <[email protected]>
jonathandw743
force-pushed
the
nodeexportnotes
branch
from
388cb60 to
d980d0d
Compare
Co-authored-by: Max Hilbrunner <[email protected]>
jonathandw743
force-pushed
the
nodeexportnotes
branch
from
d980d0d to
dcd5f30
Compare
mhilbrunner
approved these changes
Dec 16, 2024
dalexeev
reviewed
Jan 1, 2025
| Node exporting is only supported in Node-derived classes. When a Node is | ||
| saved to a PackedScene, using ``Node`` or ``Array[Node]`` will store a | ||
| reference to a Node in the scene, but using ``Array`` or ``Dictionary`` | ||
| to store Nodes will cause a copy of each Node to be created. |
There was a problem hiding this comment.
I don't think the latter is intentional behavior, it's more of an oversight. Although node export is implemented essentially as a hack.
There was a problem hiding this comment.
Yeah, that's how it works though and its something people can get tripped up on. Letting people know this is how it works is probably better than ignoring it, at least until the behaviour is changed.
There was a problem hiding this comment.
We try not to document bugs or behavior that might change without clearly indicating this. Otherwise, users might rely on the behavior and use the documentation as an argument in discussions.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added a note informing of behaviour of Node exports.