diff --git a/com.unity.render-pipelines.high-definition/CHANGELOG.md b/com.unity.render-pipelines.high-definition/CHANGELOG.md index da6e3896b24..83302d72722 100644 --- a/com.unity.render-pipelines.high-definition/CHANGELOG.md +++ b/com.unity.render-pipelines.high-definition/CHANGELOG.md @@ -676,6 +676,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0. - Fixing the internsity being applied to RTAO too early leading to unexpected results (1254626). - Fix issue that caused sky to incorrectly render when using a custom projection matrix. - Fixed null reference exception when using depth pre/post pass in shadergraph with alpha clip in the material. +- Appropriately constraint blend distance of reflection probe while editing with the inspector (case 1248931) ### Changed - Improve MIP selection for decals on Transparents diff --git a/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/HDProbeUI.Drawers.cs b/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/HDProbeUI.Drawers.cs index 3cba949c92c..81286155f03 100644 --- a/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/HDProbeUI.Drawers.cs +++ b/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/HDProbeUI.Drawers.cs @@ -233,7 +233,6 @@ public static void DrawCustomSettings(SerializedHDProbe serialized, Editor owner public static void DrawInfluenceSettings(SerializedHDProbe serialized, Editor owner) { - var provider = new TProvider(); InfluenceVolumeUI.Draw(serialized.probeSettings.influence, owner); } diff --git a/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/Volume/InfluenceVolumeUI.Drawers.cs b/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/Volume/InfluenceVolumeUI.Drawers.cs index 9e4443bbc2e..f1f9eb3f08e 100644 --- a/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/Volume/InfluenceVolumeUI.Drawers.cs +++ b/com.unity.render-pipelines.high-definition/Editor/Lighting/Reflection/Volume/InfluenceVolumeUI.Drawers.cs @@ -173,18 +173,10 @@ static void Drawer_AdvancedBlendDistance(SerializedInfluenceVolume serialized, b } else { - float scalar = editorSimplifiedModeBlendDistance.floatValue; - if (!(Mathf.Approximately(blendDistancePositive.vector3Value.x, scalar) - && Mathf.Approximately(blendDistancePositive.vector3Value.y, scalar) - && Mathf.Approximately(blendDistancePositive.vector3Value.z, scalar) - && Mathf.Approximately(blendDistanceNegative.vector3Value.x, scalar) - && Mathf.Approximately(blendDistanceNegative.vector3Value.y, scalar) - && Mathf.Approximately(blendDistanceNegative.vector3Value.z, scalar))) - { - blendDistancePositive.vector3Value = blendDistanceNegative.vector3Value = new Vector3(scalar, scalar, scalar); - serialized.Apply(); - SceneView.RepaintAll(); //update gizmo - } + var scalar = Mathf.Min(editorSimplifiedModeBlendDistance.floatValue, Mathf.Min(maxBlendDistance.x, maxBlendDistance.y, maxBlendDistance.z)); + blendDistancePositive.vector3Value = blendDistanceNegative.vector3Value = new Vector3(scalar, scalar, scalar); + serialized.Apply(); + SceneView.RepaintAll(); //update gizmo } if (serialized.editorAdvancedModeEnabled.boolValue)