Improve ChannelMixerControl callbacks

MaterialGraphProject/Assets/UnityShaderEditor/Editor/Drawing/Controls/ChannelMixerControl.cs

             buttonPanel.Add(outputButtonBlue);
 
             Add(buttonPanel);
-
+            
             var redSliderPanel = new VisualElement { name = "sliderPanel" };
             redSliderPanel.Add(new Label("R"));
             Action<float> changedRedIn = (s) => { OnChangeSlider(s, 0); };
-            m_RedInputField.OnValueChanged(OnChangeRedInputField);
+            m_RedInputField.RegisterCallback<ChangeEvent<double>, int>(OnChangeInputField, 0);
             redSliderPanel.Add(m_RedInputField);
             Add(redSliderPanel);
 
             m_GreenSlider = new Slider(m_Minimum, m_Maximum, changedGreenIn);
             greenSliderPanel.Add(m_GreenSlider);
             m_GreenInputField = new FloatField { value = m_ChannelMixer.outRed.y };
-            m_GreenInputField.OnValueChanged(OnChangeGreenInputField);
+            m_GreenInputField.RegisterCallback<ChangeEvent<double>, int>(OnChangeInputField, 1);
             greenSliderPanel.Add(m_GreenInputField);
             Add(greenSliderPanel);
 
             m_BlueSlider = new Slider(m_Minimum, m_Maximum, changedBlueIn);
             blueSliderPanel.Add(m_BlueSlider);
             m_BlueInputField = new FloatField { value = m_ChannelMixer.outRed.z };
-            m_BlueInputField.OnValueChanged(OnChangeBlueInputField);
+            m_BlueInputField.RegisterCallback<ChangeEvent<double>, int>(OnChangeInputField, 2);
             blueSliderPanel.Add(m_BlueInputField);
             Add(blueSliderPanel);
 
             m_PropertyInfo.SetValue(m_Node, m_ChannelMixer, null);
         }
 
-        void OnChangeRedInputField(ChangeEvent<double> evt)
+        void OnChangeInputField(ChangeEvent<double> evt, int inChannel)
         {
             if (!m_Initialized)
                 return;
             {
                 case 1:
-                    m_ChannelMixer.outGreen[0] = value;
+                    m_ChannelMixer.outGreen[inChannel] = value;
-                    m_ChannelMixer.outBlue[0] = value;
+                    m_ChannelMixer.outBlue[inChannel] = value;
-                    m_ChannelMixer.outRed[0] = value;
+                    m_ChannelMixer.outRed[inChannel] = value;
-            m_RedSlider.value = value;
-            m_PropertyInfo.SetValue(m_Node, m_ChannelMixer, null);
-            Dirty(ChangeType.Repaint);
-        }
-
-        void OnChangeGreenInputField(ChangeEvent<double> evt)
-        {
-            if (!m_Initialized)
-                return;
-            var value = Mathf.Max(Mathf.Min((float)evt.newValue, m_Maximum), m_Minimum);
-            m_Node.owner.owner.RegisterCompleteObjectUndo("Input Field Change");
-            switch (m_OutChannel)
+            switch(inChannel)
-                    m_ChannelMixer.outGreen[1] = value;
+                    m_GreenSlider.value = value;
-                    m_ChannelMixer.outBlue[1] = value;
+                    m_BlueSlider.value = value;
-                    m_ChannelMixer.outRed[1] = value;
+                    m_RedSlider.value = value;
-            m_GreenSlider.value = value;
-            m_PropertyInfo.SetValue(m_Node, m_ChannelMixer, null);
-            Dirty(ChangeType.Repaint);
-        }
-
-        void OnChangeBlueInputField(ChangeEvent<double> evt)
-        {
-            if (!m_Initialized)
-                return;
-            var value = Mathf.Max(Mathf.Min((float)evt.newValue, m_Maximum), m_Minimum);
-            m_Node.owner.owner.RegisterCompleteObjectUndo("Input Field Change");
-            switch (m_OutChannel)
-            {
-                case 1:
-                    m_ChannelMixer.outGreen[2] = value;
-                    break;
-                case 2:
-                    m_ChannelMixer.outBlue[2] = value;
-                    break;
-                default:
-                    m_ChannelMixer.outRed[2] = value;
-                    break;
-            }
-            m_BlueSlider.value = value;
             m_PropertyInfo.SetValue(m_Node, m_ChannelMixer, null);
             Dirty(ChangeType.Repaint);
         }