Skip to content

Conversation

@kylestarr
Copy link
Contributor

Commenting out code from the previous exercise that causes the icon to float up, making it difficult for users to see the example of directional input.

Commenting out code from the previous exercise that causes the icon to float up, making it difficult for users to see the example of directional input.
Removing unnecessary spaces in C# comments

Co-authored-by: A Thousand Ships <96648715+AThousandShips@users.noreply.github.com>
@skyace65 skyace65 added enhancement area:getting started Issues and PRs related to the Getting Started section of the documentation cherrypick:4.2 labels Jan 17, 2024
Copy link
Contributor

@skyace65 skyace65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to tell people to un-comment the Position line in the next section, otherwise that code doesn't work.

kylestarr and others added 2 commits January 25, 2024 20:21
Updating phrasing per suggestion

Co-authored-by: Matthew <matthewehr@hotmail.com>
Informing user to uncomment the commented code.

To only move when pressing a key, we need to modify the code that calculates the
velocity. Replace the line starting with ``var velocity`` with the code below.
velocity. Uncomment the code and replace the line starting with ``var velocity`` with the code below.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@skyace65 I've made a simple update that accounts for both comments.

@skyace65 skyace65 merged commit a621959 into godotengine:master Feb 4, 2024
@skyace65
Copy link
Contributor

skyace65 commented Feb 4, 2024

Thanks!

mhilbrunner pushed a commit that referenced this pull request Jul 24, 2024
Update scripting_player_input.rst

(cherry picked from commit a621959)
@mhilbrunner
Copy link
Member

Cherry-picked to 4.2 in #9648.

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 enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants