Skip to content

Conversation

@ershn
Copy link
Contributor

@ershn ershn commented Oct 5, 2023

IMO the current example isn't very high quality. This PR tries to improve it.

  • use menu consistently instead of menu/popup/screen/panel
  • fix wording that was technically incorrect/ambiguous
  • add more details to leave less room for errors
  • simplify the setup/script code
  • change method names to PascalCase in the C# code examples

@ershn ershn force-pushed the rework_pause_menu_example branch from bdca00f to c6b272e Compare October 5, 2023 15:57
@Calinou Calinou added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation cherrypick:4.1 labels Nov 9, 2023
@ershn ershn force-pushed the rework_pause_menu_example branch from 8c5b466 to fc7a41f Compare November 10, 2023 08:33
@ershn
Copy link
Contributor Author

ershn commented Nov 10, 2023

Applied review suggestions and squashed the commits.

Copy link
Member

@raulsntos raulsntos left a comment

Choose a reason for hiding this comment

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

The C# changes look good to me.

@skyace65 skyace65 merged commit daf78dd into godotengine:master Nov 24, 2023
@skyace65
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:manual Issues and PRs related to the Manual/Tutorials section of the documentation cherrypick:4.1 enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants