Skip to content

Always apply tonemapping to solid color background in Compatibility#111550

Open
Kaleb-Reid wants to merge 2 commits into
godotengine:masterfrom
Kaleb-Reid:compat-fix-bg-tonemap
Open

Always apply tonemapping to solid color background in Compatibility#111550
Kaleb-Reid wants to merge 2 commits into
godotengine:masterfrom
Kaleb-Reid:compat-fix-bg-tonemap

Conversation

@Kaleb-Reid

@Kaleb-Reid Kaleb-Reid commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

The sky shader always needs to be drawn if we are tonemapping.

If we are using a transparent background, tonemapping needs to happen in post because we are not drawing a sky shader.

Depends on #110911

Fixes #102860

Comment thread drivers/gles3/rasterizer_scene_gles3.cpp Outdated
@Kaleb-Reid Kaleb-Reid force-pushed the compat-fix-bg-tonemap branch 5 times, most recently from fdabdd0 to 1a578bf Compare October 23, 2025 04:21
@akien-mga akien-mga requested a review from clayjohn January 8, 2026 21:25
@Repiteo Repiteo modified the milestones: 4.6, 4.x Jan 26, 2026
@Kaleb-Reid Kaleb-Reid force-pushed the compat-fix-bg-tonemap branch from 1a578bf to e51dd3e Compare January 29, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenGL: Changing Scaling 3D Scale away from 1.0 breaks tonemapping

4 participants