Merge pull request #3990 from Railboy/mrtk_appbar_custom_button

Stops AppBar from changing custom button visibility

Author: wiwei

Assets/MixedRealityToolkit.SDK/Features/UX/Scripts/AppBar/AppBar.cs

@@ -256,8 +256,16 @@ private void InitializeButtons()
                     throw new Exception("Found a transform without an AppBarButton component under buttonTransforms!");
 
                 appBarButton.InitializeButtonContent(this);
-                // Set to invisible initially
-                appBarButton.SetVisible(false);
+                // Set to invisible initially if not custom
+                switch (appBarButton.ButtonType)
+                {
+                    case ButtonTypeEnum.Custom:
+                        break;
+
+                    default:
+                        appBarButton.SetVisible(false);
+                        break;
+                }
 
                 buttons.Add(appBarButton);
             }
@@ -278,11 +286,20 @@ private void UpdateButtons()
             {
                 AppBarButton button = buttons[i];
 
-                if (button.ButtonType != ButtonTypeEnum.Custom)
-                    button.SetVisible(GetButtonVisible(button.ButtonType));
+                switch (button.ButtonType)
+                {
+                    case ButtonTypeEnum.Custom:
+                        break;
+
+                    default:
+                        button.SetVisible(GetButtonVisible(button.ButtonType));
+                        break;
+                }
 
                 if (!buttons[i].Visible)
+                {
                     continue;
+                }
 
                 activeButtonNum++;
             }