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

WebGLRenderer: Disable transmission pass if override material is enabled. #26944

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

Oletus
Copy link
Contributor

@Oletus Oletus commented Oct 11, 2023

Original object materials are ignored in rendering in case an override material is used, so rendering the transmission pass is not necessary in this case.

This contribution is funded by Higharc

Original object materials are ignored in rendering in case an override material is used, so rendering the transmission pass is not necessary in this case.

This contribution is funded by https://higharc.com/
@github-actions
Copy link

📦 Bundle size

Full ESM build, minified and gzipped.

Filesize dev Filesize PR Diff
649.1 kB (160.9 kB) 649.1 kB (160.9 kB) +66 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

Filesize dev Filesize PR Diff
442.4 kB (107.1 kB) 442.5 kB (107.1 kB) +66 B
@Mugen87 Mugen87 added this to the r158 milestone Oct 11, 2023
@Mugen87 Mugen87 merged commit af36c1c into mrdoob:dev Oct 11, 2023
20 checks passed
@Mugen87 Mugen87 changed the title Disable transmission pass if override material is enabled Oct 11, 2023
@Oletus Oletus deleted the transmissive-override-material branch October 11, 2023 11:57
@Oletus
Copy link
Contributor Author

Oletus commented Oct 11, 2023

Thanks for the fast review! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants