Sg/inline color node hdr to default fix

com.unity.shadergraph/CHANGELOG.md

@@ -93,6 +93,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - Fixed a bug with the `Transform` node where converting from `Absolute World` space in a sub graph causes invalid subscript errors. [1190813](https://issuetracker.unity3d.com/issues/shadergraph-invalid-subscript-errors-are-thrown-when-connecting-a-subgraph-with-transform-node-with-unlit-master-node)
 - Fixed a bug where the `Position` node would change coordinate spaces from `World` to `Absolute World` when shaders recompile. [1184617](https://issuetracker.unity3d.com/product/unity/issues/guid/1184617/)
 - Fixed a bug where instanced shaders wouldn't compile on PS4.
+- Fixed a bug where switching a Color Nodes' Mode between Default and HDR would cause the Color to be altered incorrectly.
 - Optimized loading a large Shader Graph. [1209047](https://issuetracker.unity3d.com/issues/shader-graph-unresponsive-editor-when-using-large-graphs)
 - Fixed NaN issue in triplanar SG node when blend goes to 0.
 - Fixed an issue where Blackboard properties would not duplicate with `Precision` or `Hybrid Instancing` options. 

com.unity.shadergraph/Editor/Data/Nodes/Input/Basic/ColorNode.cs

@@ -54,22 +54,17 @@ public Color color
                 if ((value.color == m_Color.color) && (value.mode == m_Color.mode))
                     return;
 
-                if(value.mode != m_Color.mode)
+                if ((value.mode != m_Color.mode) && (value.mode == ColorMode.Default))
                 {
-                    if(value.mode == ColorMode.HDR)
-                        value.color = value.color.gamma;
-                    else
-                    {
-                        float r = Mathf.Clamp(value.color.r, 0, 1);
-                        float g = Mathf.Clamp(value.color.g, 0, 1);
-                        float b = Mathf.Clamp(value.color.b, 0, 1);
-                        float a = Mathf.Clamp(value.color.a, 0, 1);
-                        value.color = new UnityEngine.Color(r, g, b, a);
-                    }
+                    float r = Mathf.Clamp(value.color.r, 0, 1);
+                    float g = Mathf.Clamp(value.color.g, 0, 1);
+                    float b = Mathf.Clamp(value.color.b, 0, 1);
+                    float a = Mathf.Clamp(value.color.a, 0, 1);
+                    value.color = new UnityEngine.Color(r, g, b, a);
                 }
 
                 m_Color = value;
-                Dirty(ModificationScope.Node);
+                Dirty(ModificationScope.Graph);
             }
         }
 

com.unity.shadergraph/Editor/Drawing/Blackboard/BlackboardFieldPropertyView.cs

@@ -300,7 +300,7 @@ void BuildColorPropertyField(ColorShaderProperty property)
                         property.colorMode = (ColorMode)evt.newValue;
                         colorField.hdr = property.colorMode == ColorMode.HDR;
                         colorField.MarkDirtyRepaint();
-                        DirtyNodes();
+                        DirtyNodes(ModificationScope.Graph);
                     });
                 AddRow("Mode", colorModeField);
             }