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

Document support for 2D per-instance uniforms in Shading language #10749

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

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Mar 6, 2025

@Calinou Calinou added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:shaders cherrypick:4.4 labels Mar 6, 2025
Copy link
Contributor

@tetrapod00 tetrapod00 left a comment

Choose a reason for hiding this comment

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

Supersedes #10737 if merged.

This section also mentions GeometryInstance3D specifically a few places. Those should likely be amended to also mention CanvasItem too.

Comment on lines +1061 to +1063
Since Godot 4.4, per-instance uniforms are available in ``canvas_item`` (2D)
and ``spatial`` (3D) shaders. Prior to Godot 4.4, per-instance uniforms are
only available in ``spatial`` shaders.
Copy link
Contributor

@tetrapod00 tetrapod00 Mar 6, 2025

Choose a reason for hiding this comment

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

Suggested change
Since Godot 4.4, per-instance uniforms are available in ``canvas_item`` (2D)
and ``spatial`` (3D) shaders. Prior to Godot 4.4, per-instance uniforms are
only available in ``spatial`` shaders.
Since Godot 4.4, per-instance uniforms are available in both ``canvas_item`` (2D)
and ``spatial`` (3D) shaders.

I don't think we should explicitly mention older versions like this?
Or just drop the first version if the wording is ambiguous:

Suggested change
Since Godot 4.4, per-instance uniforms are available in ``canvas_item`` (2D)
and ``spatial`` (3D) shaders. Prior to Godot 4.4, per-instance uniforms are
only available in ``spatial`` shaders.
Per-instance uniforms are available in both ``canvas_item`` (2D) and ``spatial`` (3D) shaders.

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.4 enhancement topic:shaders
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants