Revert "[3D] Add gradient background to 3D scene"#65933
Revert "[3D] Add gradient background to 3D scene"#65933
Conversation
This reverts commit f011ca5.
This reverts commit c280d10.
This reverts commit 323521d.
This reverts commit a1d035c.
…be, rename camera and skybox to just camera" This reverts commit d15b34d.
This reverts commit e1924de.
This reverts commit 65c70b5.
This reverts commit 8db8bb1.
This reverts commit a0c24d7.
This reverts commit 41de612.
This reverts commit 9294316.
… the gradient and skybox entities - gui modifications, use combobox to switch between skybox and gradient background - shader mixes top bottom colors" This reverts commit 8da2a92.
🪟 Windows Qt6 buildsDownload Windows Qt6 builds of this PR for testing. 🍎 MacOS Qt6 buildsDownload MacOS Qt6 builds of this PR for testing. |
|
@wonder-sk Sorry for the hassle - let's avoid the revert here, I think we can do the cleanups as follow ups. My outstanding issues are:
|
|
Big +1 on point nb 2 :) |
|
Please keep in mind that the gradient background is not a skybox. It's supposed to be just a background to make working with the 3d view more comfortable. It does not follow the camera and should not be used to stylize the 3d view for presentation, that's what the skybox is for. My take would be to keep the gradient background as a simple 2 color gradient and move its setting to a project wide setting. To me it is a slight extension to the existing background color project setting and should be treated the same. So no per 3d view configuration of colors. In the 3d view's settings one would only pick whether he needs a skybox rendered on top or not. Bonus is we keep the project xml tidier - less items in the 3d views' xml. For stylizing a 3d scene we should guide users into using the skybox instead. I know finding and tweaking a sunset skybox is much more complicated than picking a few colors and ramp, so instead of "abusing" the background gradient for that, we could add a "dynamic skybox" option. |
|
Sorry if it hasn't been communicated clearly, but it is a static background, not a texture that is applied on the Skybox, like @nyalldawson you showed in your working branch for the color ramp approach. Screencast_20260427_121546.webm |
Reverts #65820 per Nyall's request