refactor(components): rename the 'OnToggle' event to 'OnMenuToggle' i…

…n the `NavbarContext`
LumexUI · Jul 14, 2024 · c7ec738 · c7ec738
1 parent 32d449a
commit c7ec738
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/LumexUI/Components/Navbar/LumexNavbarMenu.razor.cs b/src/LumexUI/Components/Navbar/LumexNavbarMenu.razor.cs
@@ -51,7 +51,7 @@ protected override void OnInitialized()
         ContextNullException.ThrowIfNull( Context, nameof( LumexNavbarMenu ) );
 
         Context.Register( this );
-        Context.OnToggle += StateHasChanged;
+        Context.OnMenuToggle += StateHasChanged;
     }
 
     protected override async Task OnAfterRenderAsync( bool firstRender )

diff --git a/src/LumexUI/Components/Navbar/NavbarContext.cs b/src/LumexUI/Components/Navbar/NavbarContext.cs
@@ -8,7 +8,7 @@ internal class NavbarContext( LumexNavbar owner ) : IComponentContext<LumexNavba
     public LumexNavbarMenu? Menu { get; private set; }
     public bool IsMenuExpanded { get; private set; }
 
-    public event Action? OnToggle;
+    public event Action OnMenuToggle = default!;
 
     public void Register( LumexNavbarMenu menu )
     {
@@ -21,5 +21,5 @@ public void ToggleMenu()
         NotifyStateChanged();
     }
 
-    private void NotifyStateChanged() => OnToggle?.Invoke();
+    private void NotifyStateChanged() => OnMenuToggle.Invoke();
 }