Merge pull request #79 from codersyntax/change-to-dropdown-item-event

Resolve Issue 78 : DropdownItem Click Event Parameter Type Issue
siemens · Sep 3, 2024 · b638b2b · b638b2b
2 parents 7785c04 + 9547b66
commit b638b2b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/SiemensIXBlazor.Tests/Dropdown/DropdownItemTest.cs b/SiemensIXBlazor.Tests/Dropdown/DropdownItemTest.cs
@@ -35,10 +35,10 @@ public async Task EventCallbacksAreTriggeredCorrectly()
 
         var cut = RenderComponent<DropdownItem>(parameters => parameters
             .Add(p => p.OnClickEvent,
-                EventCallback.Factory.Create<string>(this, label => isOnClickEventTriggered = true)));
+                EventCallback.Factory.Create<DropdownItem>(this, label => isOnClickEventTriggered = true)));
 
         // Act
-        await cut.Instance.OnClickEvent.InvokeAsync("testLabel");
+        await cut.Instance.OnClickEvent.InvokeAsync(cut.Instance);
 
         // Assert
         Assert.True(isOnClickEventTriggered);

diff --git a/SiemensIXBlazor/Components/Dropdown/DropdownItem.razor b/SiemensIXBlazor/Components/Dropdown/DropdownItem.razor
@@ -16,5 +16,5 @@ style="@Style"
 class="@Class"
 label="@Label" 
 value="@Value" 
-@onclick="() => Clicked(Label)">
+@onclick="() => Clicked(this)">
 </ix-dropdown-item>
diff --git a/SiemensIXBlazor/Components/Dropdown/DropdownItem.razor.cs b/SiemensIXBlazor/Components/Dropdown/DropdownItem.razor.cs
@@ -18,11 +18,11 @@ public partial class DropdownItem
         [Parameter]
         public string Value { get; set; } = string.Empty;
         [Parameter]
-        public EventCallback<string> OnClickEvent { get; set; }
+        public EventCallback<DropdownItem> OnClickEvent { get; set; }
 
-        private async void Clicked(string label)
+        private async void Clicked(DropdownItem dropdownItem)
         {
-            await OnClickEvent.InvokeAsync(label);
+            await OnClickEvent.InvokeAsync(dropdownItem);
         }
     }
 }