Merge pull request #847 from PlayEveryWare/fix/set-selected-on-update

paulhazen · web-flow · commit a266239e1d94 · 2024-08-21T14:30:04.000-07:00
fix: Do not set the selected item during each Update()
diff --git a/Assets/Scripts/StandardSamples/UI/Common/SampleMenu.cs b/Assets/Scripts/StandardSamples/UI/Common/SampleMenu.cs
@@ -211,9 +211,7 @@ protected virtual void OnEnable()
         /// </summary>
         protected virtual void Update()
         {
-            // If nothing is selected, set the selected (focused) control to be
-            // UIFirstSelected (if that field has been set).
-            SetSelected();
+            // Default behavior is to take no action.
         }
 
         /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -211,9 +211,7 @@ protected virtual void OnEnable()`
`211`	`211`	`/// </summary>`
`212`	`212`	`protected virtual void Update()`
`213`	`213`	`{`
`214`		`- // If nothing is selected, set the selected (focused) control to be`
`215`		`- // UIFirstSelected (if that field has been set).`
`216`		`- SetSelected();`
	`214`	`+ // Default behavior is to take no action.`
`217`	`215`	`}`
`218`	`216`
`219`	`217`	`/// <summary>`