refactor(demo): extract IAM buttons to SectionBuilder

fadi-george · fadi-george · commit 14daa69694af · 2026-05-25T11:55:06.000-07:00
diff --git a/examples/demo/Assets/Resources/Theme.uss b/examples/demo/Assets/Resources/Theme.uss
@@ -329,35 +329,6 @@
     margin-bottom: var(--card-gap);
 }
 
-.iam-button {
-    background-color: var(--os-primary);
-    color: white;
-    border-radius: 8px;
-    padding: 14px 16px;
-    border-width: 0;
-    font-size: 14px;
-    -unity-font-style: bold;
-    -unity-text-align: middle-left;
-    margin-bottom: var(--card-gap);
-    flex-direction: row;
-    align-items: center;
-}
-
-.iam-button-icon {
-    font-size: 18px;
-    color: white;
-    margin-right: 8px;
-    width: 20px;
-    -unity-font-definition: resource("MaterialIcons-Regular");
-    -unity-text-align: middle-center;
-}
-
-.iam-button-label {
-    color: white;
-    font-size: 14px;
-    -unity-font-style: bold;
-}
-
 .key-value-item {
     flex-direction: row;
     align-items: center;
diff --git a/examples/demo/Assets/Scripts/UI/Sections/AliasesSectionController.cs b/examples/demo/Assets/Scripts/UI/Sections/AliasesSectionController.cs
@@ -62,7 +62,7 @@ private void RefreshList()
             SectionBuilder.RenderPairList(
                 _listContainer,
                 _viewModel.Aliases,
-                "No Aliases Added",
+                "No aliases added",
                 "aliases",
                 loading: _viewModel.IsLoading
             );
diff --git a/examples/demo/Assets/Scripts/UI/Sections/EmailsSectionController.cs b/examples/demo/Assets/Scripts/UI/Sections/EmailsSectionController.cs
@@ -61,7 +61,7 @@ private void RefreshList()
                 _listContainer.Add(
                     _viewModel.IsLoading
                         ? SectionBuilder.CreateLoadingState("emails")
-                        : SectionBuilder.CreateEmptyState("No Emails Added", "emails")
+                        : SectionBuilder.CreateEmptyState("No emails added", "emails")
                 );
                 return;
             }
diff --git a/examples/demo/Assets/Scripts/UI/Sections/SendIamSectionController.cs b/examples/demo/Assets/Scripts/UI/Sections/SendIamSectionController.cs
@@ -1,7 +1,5 @@
 using System;
 using OneSignalDemo.Models;
-using OneSignalDemo.Services;
-using OneSignalDemo.UI;
 using OneSignalDemo.ViewModels;
 using UnityEngine.UIElements;
 
@@ -31,63 +29,35 @@ private VisualElement BuildSection()
             );
 
             section.Add(
-                CreateIamButton(
+                SectionBuilder.CreatePrimaryButton(
                     "TOP BANNER",
-                    MaterialIcons.VerticalAlignTop,
                     "send_iam_top_banner_button",
-                    InAppMessageType.TopBanner
+                    () => _viewModel.SendInAppMessage(InAppMessageType.TopBanner)
                 )
             );
             section.Add(
-                CreateIamButton(
+                SectionBuilder.CreatePrimaryButton(
                     "BOTTOM BANNER",
-                    MaterialIcons.VerticalAlignBottom,
                     "send_iam_bottom_banner_button",
-                    InAppMessageType.BottomBanner
+                    () => _viewModel.SendInAppMessage(InAppMessageType.BottomBanner)
                 )
             );
             section.Add(
-                CreateIamButton(
+                SectionBuilder.CreatePrimaryButton(
                     "CENTER MODAL",
-                    MaterialIcons.CropSquare,
                     "send_iam_center_modal_button",
-                    InAppMessageType.CenterModal
+                    () => _viewModel.SendInAppMessage(InAppMessageType.CenterModal)
                 )
             );
             section.Add(
-                CreateIamButton(
+                SectionBuilder.CreatePrimaryButton(
                     "FULL SCREEN",
-                    MaterialIcons.Fullscreen,
                     "send_iam_full_screen_button",
-                    InAppMessageType.FullScreen
+                    () => _viewModel.SendInAppMessage(InAppMessageType.FullScreen)
                 )
             );
 
             return section;
         }
-
-        private VisualElement CreateIamButton(
-            string text,
-            string icon,
-            string name,
-            InAppMessageType type
-        )
-        {
-            void SendIam() => _viewModel.SendInAppMessage(type);
-            var btn = new Button(SendIam);
-            btn.name = name;
-            btn.AddToClassList("iam-button");
-            AccessibilityBridge.RegisterE2ETapTarget(btn, () => btn.enabledInHierarchy, SendIam);
-
-            var iconLabel = new Label(icon);
-            iconLabel.AddToClassList("iam-button-icon");
-            btn.Add(iconLabel);
-
-            var textLabel = new Label(text);
-            textLabel.AddToClassList("iam-button-label");
-            btn.Add(textLabel);
-
-            return btn;
-        }
     }
 }
diff --git a/examples/demo/Assets/Scripts/UI/Sections/SmsSectionController.cs b/examples/demo/Assets/Scripts/UI/Sections/SmsSectionController.cs
@@ -61,7 +61,7 @@ private void RefreshList()
                 _listContainer.Add(
                     _viewModel.IsLoading
                         ? SectionBuilder.CreateLoadingState("sms")
-                        : SectionBuilder.CreateEmptyState("No SMS Added", "sms")
+                        : SectionBuilder.CreateEmptyState("No SMS added", "sms")
                 );
                 return;
             }
diff --git a/examples/demo/Assets/Scripts/UI/Sections/TagsSectionController.cs b/examples/demo/Assets/Scripts/UI/Sections/TagsSectionController.cs
@@ -71,7 +71,7 @@ private void RefreshList()
             SectionBuilder.RenderPairList(
                 _listContainer,
                 _viewModel.Tags,
-                "No Tags Added",
+                "No tags added",
                 "tags",
                 loading: _viewModel.IsLoading,
                 onRemove: key => _viewModel.RemoveTag(key)
diff --git a/examples/demo/Assets/Scripts/ViewModels/AppViewModel.cs b/examples/demo/Assets/Scripts/ViewModels/AppViewModel.cs
@@ -150,7 +150,6 @@ public void LoginUser(string externalUserId)
                 _externalUserId = externalUserId;
 
                 Debug.Log($"[{Tag}] Logged in as: {externalUserId}");
-                ShowToast($"Logged in as: {externalUserId}");
                 // The user 'change' listener runs FetchUserDataFromApi once the new
                 // onesignalId is assigned; that call clears isLoading in its finally.
             }
@@ -172,7 +171,6 @@ public void LogoutUser()
             ClearUserData();
 
             Debug.Log($"[{Tag}] Logged out");
-            ShowToast("Logged out");
             NotifyStateChanged();
         }
 

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ private void RefreshList()`
`61`	`61`	`_listContainer.Add(`
`62`	`62`	`_viewModel.IsLoading`
`63`	`63`	`? SectionBuilder.CreateLoadingState("emails")`
`64`		`- : SectionBuilder.CreateEmptyState("No Emails Added", "emails")`
	`64`	`+ : SectionBuilder.CreateEmptyState("No emails added", "emails")`
`65`	`65`	`);`
`66`	`66`	`return;`
`67`	`67`	`}`
Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,6 @@ public void LoginUser(string externalUserId)`
`150`	`150`	`_externalUserId = externalUserId;`
`151`	`151`
`152`	`152`	`Debug.Log($"[{Tag}] Logged in as: {externalUserId}");`
`153`		`- ShowToast($"Logged in as: {externalUserId}");`
`154`	`153`	`// The user 'change' listener runs FetchUserDataFromApi once the new`
`155`	`154`	`// onesignalId is assigned; that call clears isLoading in its finally.`
`156`	`155`	`}`
`@@ -172,7 +171,6 @@ public void LogoutUser()`
`172`	`171`	`ClearUserData();`
`173`	`172`
`174`	`173`	`Debug.Log($"[{Tag}] Logged out");`
`175`		`- ShowToast("Logged out");`
`176`	`174`	`NotifyStateChanged();`
`177`	`175`	`}`
`178`	`176`