Made the PathEditor queue another update loop whenever scene or inspector controls are changed. This will let external scripts react to changes in their Update loop if they have the [ExecuteAlways] or [ExecuteInEditMode] attributes added to their script.

keenanwoodall · keenanwoodall · commit 5bafe42317bc · 2019-02-05T13:34:56.000-06:00
diff --git a/PathCreator/Core/Editor/PathEditor.cs b/PathCreator/Core/Editor/PathEditor.cs
@@ -112,7 +112,6 @@ public override void OnInspectorGUI()
 
         void DrawBezierPathInspector()
         {
-
             using (var check = new EditorGUI.ChangeCheckScope())
             {
                 // Path options:
@@ -177,6 +176,7 @@ void DrawBezierPathInspector()
                 if (check.changed)
                 {
                     SceneView.RepaintAll();
+					EditorApplication.QueuePlayerLoopUpdate ();
                 }
             }
         }
@@ -196,7 +196,8 @@ void DrawVertexPathInspector()
                     {
                         data.VertexPathSettingsChanged();
                         SceneView.RepaintAll();
-                    }
+						EditorApplication.QueuePlayerLoopUpdate ();
+					}
                 }
             }
 
@@ -211,6 +212,7 @@ void DrawVertexPathInspector()
                     if (check.changed)
                     {
                         SceneView.RepaintAll();
+						EditorApplication.QueuePlayerLoopUpdate ();
                     }
                 }
                 DrawGlobalDisplaySettingsInspector();
@@ -231,7 +233,7 @@ void DrawGlobalDisplaySettingsInspector()
                 {
                     UpdateGlobalDisplaySettings();
                     SceneView.RepaintAll();
-                }
+				}
             }
         }
 
@@ -241,25 +243,33 @@ void DrawGlobalDisplaySettingsInspector()
 
         void OnSceneGUI()
         {
-            handlesStartCol = Handles.color;
-            switch (data.tabIndex)
-            {
-                case bezierPathTab:
-                    ProcessBezierPathInput(Event.current);
-                    DrawBezierPathSceneEditor();
-                    break;
-                case vertexPathTab:
-                    DrawVertexPathSceneEditor();
-                    break;
-            }
-
-
-            // Don't allow clicking over empty space to deselect the object
-            if (Event.current.type == EventType.Layout)
-            {
-                HandleUtility.AddDefaultControl(0);
-            }
-        }
+			using (var check = new EditorGUI.ChangeCheckScope ())
+			{
+				handlesStartCol = Handles.color;
+				switch (data.tabIndex)
+				{
+					case bezierPathTab:
+						ProcessBezierPathInput (Event.current);
+						DrawBezierPathSceneEditor ();
+						break;
+					case vertexPathTab:
+						DrawVertexPathSceneEditor ();
+						break;
+				}
+
+
+				// Don't allow clicking over empty space to deselect the object
+				if (Event.current.type == EventType.Layout)
+				{
+					HandleUtility.AddDefaultControl (0);
+				}
+
+				if (check.changed)
+				{
+					EditorApplication.QueuePlayerLoopUpdate ();
+				}
+			}
+		}
 
         void DrawVertexPathSceneEditor()
         {

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,6 @@ public override void OnInspectorGUI()`
`112`	`112`
`113`	`113`	`void DrawBezierPathInspector()`
`114`	`114`	`{`
`115`		`-`
`116`	`115`	`using (var check = new EditorGUI.ChangeCheckScope())`
`117`	`116`	`{`
`118`	`117`	`// Path options:`
`@@ -177,6 +176,7 @@ void DrawBezierPathInspector()`
`177`	`176`	`if (check.changed)`
`178`	`177`	`{`
`179`	`178`	`SceneView.RepaintAll();`
	`179`	`+ EditorApplication.QueuePlayerLoopUpdate ();`
`180`	`180`	`}`
`181`	`181`	`}`
`182`	`182`	`}`
`@@ -196,7 +196,8 @@ void DrawVertexPathInspector()`
`196`	`196`	`{`
`197`	`197`	`data.VertexPathSettingsChanged();`
`198`	`198`	`SceneView.RepaintAll();`
`199`		`- }`
	`199`	`+ EditorApplication.QueuePlayerLoopUpdate ();`
	`200`	`+ }`
`200`	`201`	`}`
`201`	`202`	`}`
`202`	`203`
`@@ -211,6 +212,7 @@ void DrawVertexPathInspector()`
`211`	`212`	`if (check.changed)`
`212`	`213`	`{`
`213`	`214`	`SceneView.RepaintAll();`
	`215`	`+ EditorApplication.QueuePlayerLoopUpdate ();`
`214`	`216`	`}`
`215`	`217`	`}`
`216`	`218`	`DrawGlobalDisplaySettingsInspector();`
`@@ -231,7 +233,7 @@ void DrawGlobalDisplaySettingsInspector()`
`231`	`233`	`{`
`232`	`234`	`UpdateGlobalDisplaySettings();`
`233`	`235`	`SceneView.RepaintAll();`
`234`		`- }`
	`236`	`+ }`
`235`	`237`	`}`
`236`	`238`	`}`
`237`	`239`
`@@ -241,25 +243,33 @@ void DrawGlobalDisplaySettingsInspector()`
`241`	`243`
`242`	`244`	`void OnSceneGUI()`
`243`	`245`	`{`
`244`		`- handlesStartCol = Handles.color;`
`245`		`- switch (data.tabIndex)`
`246`		`- {`
`247`		`- case bezierPathTab:`
`248`		`- ProcessBezierPathInput(Event.current);`
`249`		`- DrawBezierPathSceneEditor();`
`250`		`- break;`
`251`		`- case vertexPathTab:`
`252`		`- DrawVertexPathSceneEditor();`
`253`		`- break;`
`254`		`- }`
`255`		`-`
`256`		`-`
`257`		`- // Don't allow clicking over empty space to deselect the object`
`258`		`- if (Event.current.type == EventType.Layout)`
`259`		`- {`
`260`		`- HandleUtility.AddDefaultControl(0);`
`261`		`- }`
`262`		`- }`
	`246`	`+ using (var check = new EditorGUI.ChangeCheckScope ())`
	`247`	`+ {`
	`248`	`+ handlesStartCol = Handles.color;`
	`249`	`+ switch (data.tabIndex)`
	`250`	`+ {`
	`251`	`+ case bezierPathTab:`
	`252`	`+ ProcessBezierPathInput (Event.current);`
	`253`	`+ DrawBezierPathSceneEditor ();`
	`254`	`+ break;`
	`255`	`+ case vertexPathTab:`
	`256`	`+ DrawVertexPathSceneEditor ();`
	`257`	`+ break;`
	`258`	`+ }`
	`259`	`+`
	`260`	`+`
	`261`	`+ // Don't allow clicking over empty space to deselect the object`
	`262`	`+ if (Event.current.type == EventType.Layout)`
	`263`	`+ {`
	`264`	`+ HandleUtility.AddDefaultControl (0);`
	`265`	`+ }`
	`266`	`+`
	`267`	`+ if (check.changed)`
	`268`	`+ {`
	`269`	`+ EditorApplication.QueuePlayerLoopUpdate ();`
	`270`	`+ }`
	`271`	`+ }`
	`272`	`+ }`
`263`	`273`
`264`	`274`	`void DrawVertexPathSceneEditor()`
`265`	`275`	`{`