Skip to content

Conversation

@Jackerty
Copy link
Contributor

@Jackerty Jackerty commented Jun 3, 2023

A node that is loaded through GDExtension can have a custom icon. Icon's path can be added via the gdextension file. (See godotengine/godot#75472).

Commit makes a mention of this in the C++ tutorial.

@paddy-exe paddy-exe added the area:manual Issues and PRs related to the Manual/Tutorials section of the documentation label Jun 4, 2023
@YuriSizov YuriSizov requested a review from paddy-exe June 4, 2023 18:36
Copy link
Contributor

@paddy-exe paddy-exe left a comment

Choose a reason for hiding this comment

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

Thanks for your PR! I reworked the sentence structure to clarify a few things and shorten it the important bits. Furthermore, the path should point to a 16 by 16 px image so the icon is scaled properly so I emphasized this. I hope you can agree with my suggestions :)

A node that is loaded through GDExtension can have a custom icon. Icon's path can be added via the `gdextension` file.
(See godotengine/godot#75472).

Commit makes a mention of this in the C++ tutorial.
Copy link
Contributor

@paddy-exe paddy-exe left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for the quick updates!

@mhilbrunner mhilbrunner merged commit a34243c into godotengine:master Jul 18, 2023
@mhilbrunner
Copy link
Member

Thanks for contributing and congrats on your first merged PR to Godot's docs! Sorry this took a bit to get merged as I was a bit busy due to real life events and the Valencia sprint :)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants