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

Add info to turn off preview sunlight before working with 3D shader lighting in 'Your first 3D shader'. #9897

Merged
merged 2 commits into from
Sep 5, 2024

Conversation

FarzadBT
Copy link
Contributor

@FarzadBT FarzadBT commented Sep 3, 2024

By default in Godot 4.3, preview sunlight is enabled and only disabled by a DirectionalLight3D being added to the scene. If you follow the guide and add an OmniLight3D, the preview sunlight stays enabled and causes a drastically different look compared to the images in the guide.

With preview sunlight (default).
image

Without preview sunlight.
image

Optionally also add an image to point out the toggle button;
image

…lighting.

By default in Godot 4.3, preview sunlight is enabled and only disabled by a DirectionalLight3D being added to the scene. If you follow the guide and add an OmniLight3D, the preview sunlight stays enabled and causes a drastically different look.
@AThousandShips AThousandShips added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:shaders labels Sep 3, 2024
Co-authored-by: A Thousand Ships <[email protected]>
@skyace65 skyace65 merged commit 52eefaf into godotengine:master Sep 5, 2024
1 check passed
@skyace65
Copy link
Contributor

skyace65 commented Sep 5, 2024

Thanks! And congrats on your first merged PR!

@FarzadBT FarzadBT deleted the FarzadBT-patch-1 branch September 5, 2024 08:29
mhilbrunner pushed a commit that referenced this pull request Oct 4, 2024
 Add info to turn off preview sunlight before working with 3D shader lighting in 'Your first 3D shader'.
@mhilbrunner
Copy link
Member

Cherry-picked to 4.3 in #10038.

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

Successfully merging this pull request may close these issues.

4 participants