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

Mentioning the GDExtension icon support #7461

Merged
merged 1 commit into from
Jul 18, 2023

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