Fix ClippingPrimitive.ResetRenderer clearing all MaterialPropertyBlock properties (#244)

Cameron-Micka · web-flow · commit 7d9f9160d8c6 · 2026-03-03T09:44:52.000-08:00
diff --git a/com.microsoft.mrtk.graphicstools.unity/Runtime/Clipping/ClippingPrimitive.cs b/com.microsoft.mrtk.graphicstools.unity/Runtime/Clipping/ClippingPrimitive.cs
@@ -182,10 +182,6 @@ public void ResetRenderer(Renderer _renderer, bool autoDestroyMaterial = true)
                 {
                     materialInstance.ReleaseMaterial(this, autoDestroyMaterial);
                 }
-
-                // Reset the material property block.
-                materialPropertyBlock.Clear();
-                _renderer.SetPropertyBlock(materialPropertyBlock);
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -182,10 +182,6 @@ public void ResetRenderer(Renderer _renderer, bool autoDestroyMaterial = true)`
`182`	`182`	`{`
`183`	`183`	`materialInstance.ReleaseMaterial(this, autoDestroyMaterial);`
`184`	`184`	`}`
`185`		`-`
`186`		`- // Reset the material property block.`
`187`		`- materialPropertyBlock.Clear();`
`188`		`- _renderer.SetPropertyBlock(materialPropertyBlock);`
`189`	`185`	`}`
`190`	`186`	`}`
`191`	`187`