Process review comments

microsoft · Jun 12, 2024 · e68fc89 · e68fc89
1 parent c8d0637
commit e68fc89
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/examples/Demo/Shared/Microsoft.FluentUI.AspNetCore.Components.xml b/examples/Demo/Shared/Microsoft.FluentUI.AspNetCore.Components.xml
@@ -6359,6 +6359,12 @@
             Adjust the vertical spacing between navlinks.
             </summary>
         </member>
+        <member name="P:Microsoft.FluentUI.AspNetCore.Components.FluentNavMenu.CustomToggle">
+            <summary>
+            Gets or sets if a custom toggle for showing/hiding the menu is used.
+            This is primarily intended to be used in a mobile view
+            </summary>
+        </member>
         <member name="P:Microsoft.FluentUI.AspNetCore.Components.FluentNavMenu.NavigationManager">
             <summary>
             Navigation manager

diff --git a/src/Core/Components/NavMenu/FluentNavBase.cs b/src/Core/Components/NavMenu/FluentNavBase.cs
@@ -95,10 +95,10 @@ public abstract class FluentNavBase : FluentComponentBase
     public string CustomToggleId { get; set; } = "navmenu-toggle";
 
     [CascadingParameter]
-    public FluentNavMenu Owner { get; set; } = default!;
+    internal FluentNavMenu Owner { get; set; } = default!;
 
     [CascadingParameter]
-    public FluentMenu? SubMenu { get; set; }
+    internal FluentMenu? SubMenu { get; set; }
 
     [Inject]
     private NavigationManager NavigationManager { get; set; } = default!;

diff --git a/src/Core/Components/NavMenu/FluentNavMenu.razor.cs b/src/Core/Components/NavMenu/FluentNavMenu.razor.cs
@@ -68,6 +68,10 @@ public partial class FluentNavMenu : FluentComponentBase
     [Parameter]
     public string? Margin { get; set; }
 
+    /// <summary>
+    /// Gets or sets if a custom toggle for showing/hiding the menu is used.
+    /// This is primarily intended to be used in a mobile view
+    /// </summary>
     [Parameter]
     public bool CustomToggle { get; set; } = false;