fix: enabledNonBlocking strategy for routes (#115)

apps/demo/src/app/app-routing.module.ts

@@ -50,7 +50,12 @@ const routes: Routes = [
 ];
 
 @NgModule({
-  imports: [RouterModule.forRoot(routes, { relativeLinkResolution: 'legacy' })],
+  imports: [
+    RouterModule.forRoot(routes, {
+      relativeLinkResolution: 'legacy',
+      initialNavigation: 'enabledBlocking',
+    }),
+  ],
   exports: [RouterModule],
 })
 export class AppRoutingModule {}

libs/xng-breadcrumb/src/lib/breadcrumb.service.ts

@@ -64,15 +64,15 @@ export class BreadcrumbService {
   private detectRouteChanges() {
     // Special case where breadcrumb service & component instantiates after a route is navigated.
     // Ex: put breadcrumbs within *ngIf and this.router.events would be empty
-    if (this.router.navigated) {
-      this.setupBreadcrumbs(this.activatedRoute.snapshot);
-    }
+    // This check is also required where  { initialNavigation: 'enabledBlocking' } is applied to routes
+    this.setupBreadcrumbs(this.activatedRoute.snapshot);
 
     this.router.events
       .pipe(filter((event) => event instanceof GuardsCheckEnd))
       .subscribe((event) => {
         // activatedRoute doesn't carry data when shouldReuseRoute returns false
-        // use the event data with RoutesRecognized as workaround
+        // use the event data with GuardsCheckEnd as workaround
+        // Check for shouldActivate in case where the authGuard returns false the breadcrumbs shouldn't be changed
         if (event instanceof GuardsCheckEnd && event.shouldActivate) {
           this.setupBreadcrumbs(event.state.root);
         }

todo.md

@@ -8,7 +8,3 @@ Why 4+ items? this limit just “feels” right. 🙃
 2. Put the ellipses as close to the first item as possible — Items that are closer to the final item (current page) hold more weight to the user so you want to make those items visible.
 3. Do not make the ellipses the actual first item — It is beneficial for users to know the origin of the current page they are on, especially if they are taken directly to the page they are on via a link.
 4. Upon refreshing / re-navigating to the page, breadcrumbs should be collapsed again — There is no clean way to manually collapse breadcrumbs (nor do I think it is worth the effort) so make sure it is collapsed by default, even if they were expanded before.
-
-## provide documentation via gitbook
-
-[gitbook](https://app.gitbook.com/@udayvunnam/s/angular/)