@sinequa/atomic-angular 1.0.13 → 1.0.15

package/fesm2022/sinequa-atomic-angular.mjs

@@ -1631,18 +1631,21 @@ async function withBootstrapApp(applicationService, { createRoutes = true }) {
     return new Promise(resolve => {
         // Check if the user is authenticated
         signIn()
-            .then((response) => {
+            .then(async (response) => {
             if (response) {
                 info('User authenticated, initializing application...');
-                // Initialize the application
-                applicationService
-                    .initialize(createRoutes)
-                    .then(() => {
+                // Initialize the application.
+                // Awaited so the APP_INITIALIZER does not resolve (and bootstrap does not
+                // complete) until initialization is done. Otherwise routed components render
+                // and set their page title before `initialize()` runs `setGeneralApp()`, which
+                // would then overwrite the page title with the bare application name.
+                try {
+                    await applicationService.initialize(createRoutes);
                     info(`Application initialized with routes: ${createRoutes} successfully.`);
-                })
-                    .catch(err => {
+                }
+                catch (err) {
                     error(`Error initializing application with routes: ${createRoutes}:`, err);
-                });
+                }
             }
             else {
                 info('User not authenticated, skipping application initialization.');
@@ -3730,10 +3733,10 @@ class ApplicationService {
         // If general is not defined or is an empty object, do nothing
         if (!general || (typeof general === "object" && Object.keys(general).length === 0))
             return;
-        if (general.name) {
-            info("Setting document title to:", general.name);
-            this.titleService.setTitle(general.name);
-        }
+        // NB: the document title is intentionally NOT set here. The page-specific title is
+        // owned by the routed components/layouts via `setTitle()`. Setting it here would
+        // overwrite the page title with the bare application name during bootstrap.
+        // The initial title falls back to the static <title> defined in index.html.
         const { light, dark, alt } = general.logo || {};
         document.documentElement.style.setProperty("--logo-alt-text", `'${alt || general.name}'`);
         // light mode logo configuration
@@ -7213,6 +7216,7 @@ class NavbarTabsComponent {
     class = input(...(ngDevMode ? [undefined, { debugName: "class" }] : []));
     router = inject(Router);
     route = inject(ActivatedRoute);
+    appStore = inject(AppStore);
     queryParamsStore = inject(QueryParamsStore);
     // Injecting the QueryService to access last search results
     queryService = inject(QueryService);
@@ -7280,6 +7284,37 @@ class NavbarTabsComponent {
     }, ...(ngDevMode ? [{ debugName: "tabs" }] : []));
     moreTabs = computed(() => this.tabs().slice(this.visibleTabCount()), ...(ngDevMode ? [{ debugName: "moreTabs" }] : []));
     changeTab() { }
+    // Computed signal to get the persistFiltersAcrossTabs value from AppStore
+    persistFiltersAcrossTabs = computed(() => !!this.appStore.general()?.features?.persistFiltersAcrossTabs, ...(ngDevMode ? [{ debugName: "persistFiltersAcrossTabs" }] : []));
+    // Determine how query params are applied on tab change.
+    // - persist: 'merge' so the tab identity params (n/t/q) are updated while
+    //   the filter params (f/sort/id/page) already in the URL are kept.
+    // - default: 'replace' so everything is rewritten from scratch.
+    getQueryParamsHandling() {
+        return this.persistFiltersAcrossTabs() ? 'merge' : 'replace';
+    }
+    // Get query params conditionally
+    getQueryParams(tab) {
+        if (this.persistFiltersAcrossTabs()) {
+            // When preserving filters, still update the tab identity params so the
+            // store rebuilds the query for the new tab. 'merge' keeps f/sort/id/page.
+            return {
+                n: tab.queryName,
+                q: this.searchText(),
+                t: tab.wsQueryTab
+            };
+        }
+        // When replacing, explicitly set the filters to undefined to clear them
+        return {
+            n: tab.queryName,
+            q: this.searchText(),
+            t: tab.wsQueryTab,
+            f: undefined,
+            sort: undefined,
+            id: undefined,
+            page: undefined
+        };
+    }
     static ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "20.3.18", ngImport: i0, type: NavbarTabsComponent, deps: [], target: i0.ɵɵFactoryTarget.Component });
     static ɵcmp = i0.ɵɵngDeclareComponent({ minVersion: "17.0.0", version: "20.3.18", type: NavbarTabsComponent, isStandalone: true, selector: "navbar-tabs", inputs: { class: { classPropertyName: "class", publicName: "class", isSignal: true, isRequired: false, transformFunction: null }, showCount: { classPropertyName: "showCount", publicName: "showCount", isSignal: true, isRequired: false, transformFunction: null }, noTruncate: { classPropertyName: "noTruncate", publicName: "noTruncate", isSignal: true, isRequired: false, transformFunction: null }, minTabWidth: { classPropertyName: "minTabWidth", publicName: "minTabWidth", isSignal: true, isRequired: false, transformFunction: null }, path: { classPropertyName: "path", publicName: "path", isSignal: true, isRequired: false, transformFunction: null } }, host: { properties: { "class": "cn('block', class())" } }, ngImport: i0, template: `
     <!-- do not display the tabs if there are no tabs -->
@@ -7299,9 +7334,10 @@ class NavbarTabsComponent {
               [attr.disabled]="showCount() && tab.count === 0 ? '' : null"
               [active]="this.currentPath() === tab.path"
               [routerLink]="[tab.routerLink]"
-              [queryParams]="{ n: tab.queryName, q: searchText(), t: tab.wsQueryTab, f: undefined, sort: undefined, id: undefined, page: undefined }"
+              [queryParams]="getQueryParams(tab)"
+              [queryParamsHandling]="getQueryParamsHandling()"
               (click)="changeTab()"
-              (keydown.enter)="router.navigate([tab.routerLink], { queryParams: { n: tab.queryName, q: searchText(), t: tab.wsQueryTab, f: undefined, sort: undefined, id: undefined, page: undefined } })"
+              (keydown.enter)="router.navigate([tab.routerLink], { queryParams: getQueryParams(tab), queryParamsHandling: getQueryParamsHandling() })"
               >
               <div [class]="cn('flex items-center content-start w-full gap-1', !noTruncate() && 'overflow-hidden min-w-0')">
               @if (tab.icon) {
@@ -7335,7 +7371,8 @@ class NavbarTabsComponent {
                 <a
                   class="inline-block whitespace-nowrap first-letter:capitalize"
                   [routerLink]="[tab.routerLink]"
-                  [queryParams]="{ n: tab.queryName, q: searchText(), t: tab.wsQueryTab, f: undefined, sort: undefined, id: undefined, page: undefined }"
+                  [queryParams]="getQueryParams(tab)"
+                  [queryParamsHandling]="getQueryParamsHandling()"
                   [attr.aria-selected]="this.currentPath() === tab.path"
                   [attr.aria-label]="tab.display | syslang | transloco"
                   (click)="changeTab()">
@@ -7393,9 +7430,10 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "20.3.18", ngImpo
               [attr.disabled]="showCount() && tab.count === 0 ? '' : null"
               [active]="this.currentPath() === tab.path"
               [routerLink]="[tab.routerLink]"
-              [queryParams]="{ n: tab.queryName, q: searchText(), t: tab.wsQueryTab, f: undefined, sort: undefined, id: undefined, page: undefined }"
+              [queryParams]="getQueryParams(tab)"
+              [queryParamsHandling]="getQueryParamsHandling()"
               (click)="changeTab()"
-              (keydown.enter)="router.navigate([tab.routerLink], { queryParams: { n: tab.queryName, q: searchText(), t: tab.wsQueryTab, f: undefined, sort: undefined, id: undefined, page: undefined } })"
+              (keydown.enter)="router.navigate([tab.routerLink], { queryParams: getQueryParams(tab), queryParamsHandling: getQueryParamsHandling() })"
               >
               <div [class]="cn('flex items-center content-start w-full gap-1', !noTruncate() && 'overflow-hidden min-w-0')">
               @if (tab.icon) {
@@ -7429,7 +7467,8 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "20.3.18", ngImpo
                 <a
                   class="inline-block whitespace-nowrap first-letter:capitalize"
                   [routerLink]="[tab.routerLink]"
-                  [queryParams]="{ n: tab.queryName, q: searchText(), t: tab.wsQueryTab, f: undefined, sort: undefined, id: undefined, page: undefined }"
+                  [queryParams]="getQueryParams(tab)"
+                  [queryParamsHandling]="getQueryParamsHandling()"
                   [attr.aria-selected]="this.currentPath() === tab.path"
                   [attr.aria-label]="tab.display | syslang | transloco"
                   (click)="changeTab()">