🐛 fix(Tab): matching route is not highlighted correctly (#611)

Author: capdiem

src/Component/BlazorComponent/Components/ItemGroup/BRoutableGroupItem.razor.cs

@@ -44,19 +44,22 @@ public BRoutableGroupItem(GroupType groupType, string defaultTag = "div") : base
 
     protected override async Task OnInitializedAsync()
     {
-        await base.OnInitializedAsync();
+        if (IsRoutable)
+        {
+            Router = new Router(this);
+            Value = Href;
+            await UpdateActiveForRoutable();
+            NavigationManager.LocationChanged += OnLocationChanged;
+        }
 
-        Router = new Router(this);
-        await UpdateActiveForRoutable();
-        NavigationManager.LocationChanged += OnLocationChanged;
+        await base.OnInitializedAsync();
     }
 
     protected override void OnParametersSet()
     {
         base.OnParametersSet();
 
         Router = new Router(this);
-
         (Tag, Attributes) = Router.GenerateRouteLink();
     }
 
@@ -71,7 +74,7 @@ private async void OnLocationChanged(object? sender, LocationChangedEventArgs e)
 
     private async Task<bool> UpdateActiveForRoutable()
     {
-        if (!IsRoutable || Router is null) return false;
+        if (Router is null) return false;
 
         var isActive = InternalIsActive;
 
@@ -89,8 +92,11 @@ private async Task<bool> UpdateActiveForRoutable()
 
     protected override ValueTask DisposeAsyncCore()
     {
-        NavigationManager.LocationChanged -= OnLocationChanged;
+        if (IsRoutable)
+        {
+            NavigationManager.LocationChanged -= OnLocationChanged;
+        }
 
         return base.DisposeAsyncCore();
     }
-}
+}

src/Component/BlazorComponent/Components/ItemGroup/ItemGroupBase.cs

@@ -72,7 +72,7 @@ private void RefreshItemsState()
             Items.ForEach(item => item.RefreshState());
         }
 
-        protected virtual StringNumber InitDefaultItemValue()
+        protected StringNumber InitDefaultItemValue()
         {
             return _registeredItemsIndex++;
         }

src/Component/BlazorComponent/Components/Tabs/BTab.razor.cs

@@ -6,8 +6,7 @@ public BTab() : base(GroupType.SlideGroup)
         {
         }
 
-        [CascadingParameter]
-        public BTabs? Tabs { get; set; }
+        [CascadingParameter] public BTabs? Tabs { get; set; }
 
         protected override bool HasRoutableAncestor => Tabs?.Routable is true;
 
@@ -17,12 +16,15 @@ public BTab() : base(GroupType.SlideGroup)
 
         private async Task HandleOnClick(MouseEventArgs args)
         {
-            await ToggleAsync();
+            if (!IsRoutable)
+            {
+                await ToggleAsync();
+            }
 
             if (OnClick.HasDelegate)
             {
                 await OnClick.InvokeAsync(args);
             }
         }
     }
-}
+}

src/Component/BlazorComponent/Mixins/Groupable/BGroupable.cs

@@ -93,16 +93,16 @@ protected BGroupable(GroupType groupType, bool bootable) : this(groupType)
 
         protected virtual bool HasTransition { get; } = false;
 
-        protected override void OnInitialized()
+        protected override async Task OnInitializedAsync()
         {
-            base.OnInitialized();
-
             if (!Matched) return;
 
             if (this is IGroupable item)
             {
                 ItemGroup!.Register(item);
             }
+
+            await base.OnInitializedAsync();
         }
 
         protected override async Task OnParametersSetAsync()