Skip to content

Conversation

@aardgoose
Copy link
Contributor

allows ordering of rendering background to be determined.

renderer.backgroundRenderOrder must be set before first render call.

(we could change this to allow dynamic ordering but that seems excessive)

This should fix #15304 :

renderer.backgroundRenderOrder = -Infinity // background will be rendered first

@mrdoob mrdoob added this to the r100 milestone Nov 28, 2018
@mrdoob
Copy link
Owner

mrdoob commented Nov 28, 2018

I'm not sure being able to define when the background renders would be such a common thing that it requires it's own API...

In the meantime I've done the order change directly in #15329

@Mugen87
Copy link
Collaborator

Mugen87 commented Nov 29, 2018

I vote to close this PR. If there are more users requesting such a feature, we can reopen it and think about an appropriate API.

@aardgoose
Copy link
Contributor Author

no problem, It's only recently that both background types have been actually on the render list to make this possible.

@aardgoose aardgoose closed this Nov 29, 2018
@mrdoob mrdoob removed this from the r100 milestone Nov 30, 2018
@aardgoose aardgoose deleted the background-order branch June 12, 2019 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

if scene use skybox , then mesh material disable depthTest and depthWrite no show

3 participants