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

Updated step 3 in the scene setup section #9337

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fanelenan7
Copy link

I'm very new to Godot, and as a good first step I have decided to follow the official docs for guidance. When I got to the Signals portion, I have followed everything as closely as possible but still ended up getting errors. Turns out step 3 was incorrect - perhaps it was correct at some point but it wasn't updated. Instead of dragging the sprite_2d.tscn on top of Node2D you needed to put it under. That cost me an hour of pulling my hair out. In the end, good folks in the Discord channel helped me figure it out. But I figured I might as well fix this so someone else who is just starting with Godot doesn't spend an hour thinking they are bad at programming.

@Calinou Calinou added enhancement area:getting started Issues and PRs related to the Getting Started section of the documentation cherrypick:4.2 labels May 12, 2024
@Calinou
Copy link
Member

Calinou commented May 12, 2024

Thanks for opening a pull request!

Unfortunately, I'm not sure if this is the right way to avoid confusion. It might be better to point users towards using the manual approach to instancing a subscene (e.g. with the chainlink icon in the Scene tree dock).

If your scene currently only has a single node, drag-and-dropping another scene or node on top is identical to drag-and-dropping it under.

The visual drag-and-drop indicator doesn't make this very clear, as it barely has any visual difference between drag-and-dropping on top of a node (which adds as a child) and below a node (which adds as a sibling). However, in scenes with only a single node, this has no difference in practice because you can't have multiple root nodes in a scene.

Also, the screenshot was made with a non-default theme and scale. Please use the default editor theme and 100% editor scale for screenshots (or downsize the screenshot in an image editor if taken at an editor scale above 100%).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:getting started Issues and PRs related to the Getting Started section of the documentation cherrypick:4.2 enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants