ps-helix 6.2.0 → 6.2.1

package/README.md

@@ -2,7 +2,7 @@
 
 A comprehensive Angular component library built with Angular 22+ featuring modern design patterns, accessibility-first development, and optimal developer experience.
 
-[![npm version](https://img.shields.io/badge/npm-6.2.0-blue.svg)](https://www.npmjs.com/package/ps-helix)
+[![npm version](https://img.shields.io/badge/npm-6.2.1-blue.svg)](https://www.npmjs.com/package/ps-helix)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Angular](https://img.shields.io/badge/Angular-22.0.0-red.svg)](https://angular.dev/)
 [![TypeScript](https://img.shields.io/badge/TypeScript-6.0.0-blue.svg)](https://www.typescriptlang.org/)
@@ -106,7 +106,7 @@ After installation, verify that ps-helix is in your `package.json`:
 ```json
 {
   "dependencies": {
-    "ps-helix": "^6.2.0"
+    "ps-helix": "^6.2.1"
   }
 }
 ```
@@ -1182,7 +1182,7 @@ Copyright (c) 2025 PACK Solutions
 
 ---
 
-**Version**: 6.2.0
+**Version**: 6.2.1
 **Built with**: Angular 22.0.0, TypeScript 6.0.0, Phosphor Icons 2.0.3
 **Author**: Fabrice PEREZ | Design Engineer at PACK Solutions
 **Last Updated**: January 2026

package/fesm2022/ps-helix.mjs

@@ -1,5 +1,5 @@
 import * as i0 from '@angular/core';
-import { input, output, computed, ChangeDetectionStrategy, Component, model, inject, ElementRef, signal, PLATFORM_ID, ViewEncapsulation, InjectionToken, viewChild, effect, Injectable, DestroyRef, Directive, ViewContainerRef, ChangeDetectorRef, ViewChild, Injector, afterNextRender, HostListener, Renderer2, contentChild, linkedSignal, EventEmitter, Output, Input, contentChildren } from '@angular/core';
+import { input, output, computed, ChangeDetectionStrategy, Component, model, inject, ElementRef, signal, PLATFORM_ID, ViewEncapsulation, InjectionToken, viewChild, effect, Injectable, DestroyRef, Directive, ViewContainerRef, ChangeDetectorRef, ViewChild, Injector, afterNextRender, HostListener, Renderer2, contentChild, linkedSignal, isDevMode, EventEmitter, Output, Input, contentChildren } from '@angular/core';
 import * as i1 from '@angular/common';
 import { CommonModule, DOCUMENT, isPlatformBrowser, NgTemplateOutlet } from '@angular/common';
 import { NG_VALUE_ACCESSOR, FormsModule } from '@angular/forms';
@@ -3505,16 +3505,31 @@ class PshPaginationComponent {
         this.itemsPerPageInput = input(10, { ...(ngDevMode ? { debugName: "itemsPerPageInput" } : /* istanbul ignore next */ {}), alias: 'itemsPerPage' });
         this.itemsPerPage = linkedSignal(this.itemsPerPageInput, /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "itemsPerPage" }] : /* istanbul ignore next */ []));
+        /**
+         * Number of pages used for rendering and navigation: never below 1, and never
+         * written back to the `totalPages` input. `totalPages` is derived data owned by
+         * the caller (`Math.ceil(total / pageSize)`, or a server-side page count), so the
+         * component reads it and clamps its own view of it instead of correcting it.
+         */
+        this.effectiveTotalPages = computed(() => {
+            const total = this.totalPages();
+            return Number.isInteger(total) && total >= 1 ? total : 1;
+        }, /* @ts-ignore */
+        ...(ngDevMode ? [{ debugName: "effectiveTotalPages" }] : /* istanbul ignore next */ []));
         this.size = input(this.config.size ?? 'medium', { ...(ngDevMode ? { debugName: "size" } : /* istanbul ignore next */ {}), transform: (value) => {
                 if (!['small', 'medium', 'large'].includes(value)) {
-                    console.warn(`[psh-pagination] Invalid size "${value}", falling back to "medium"`);
+                    if (isDevMode()) {
+                        console.warn(`[psh-pagination] Invalid size "${value}", falling back to "medium"`);
+                    }
                     return 'medium';
                 }
                 return value;
             } });
         this.variant = input(this.config.variant ?? 'default', { ...(ngDevMode ? { debugName: "variant" } : /* istanbul ignore next */ {}), transform: (value) => {
                 if (!['default', 'outline'].includes(value)) {
-                    console.warn(`[psh-pagination] Invalid variant "${value}", falling back to "default"`);
+                    if (isDevMode()) {
+                        console.warn(`[psh-pagination] Invalid variant "${value}", falling back to "default"`);
+                    }
                     return 'default';
                 }
                 return value;
@@ -3559,7 +3574,7 @@ class PshPaginationComponent {
             const pages = [];
             const halfVisible = Math.floor(this.maxVisiblePages() / 2);
             let start = Math.max(1, this.currentPage() - halfVisible);
-            const end = Math.min(this.totalPages(), start + this.maxVisiblePages() - 1);
+            const end = Math.min(this.effectiveTotalPages(), start + this.maxVisiblePages() - 1);
             if (end - start + 1 < this.maxVisiblePages()) {
                 start = Math.max(1, end - this.maxVisiblePages() + 1);
             }
@@ -3571,60 +3586,70 @@ class PshPaginationComponent {
         ...(ngDevMode ? [{ debugName: "pages" }] : /* istanbul ignore next */ []));
         this.state = computed(() => this.getState(), /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "state" }] : /* istanbul ignore next */ []));
-        this.canGoNext = computed(() => this.currentPage() < this.totalPages(), /* @ts-ignore */
+        this.canGoNext = computed(() => this.currentPage() < this.effectiveTotalPages(), /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "canGoNext" }] : /* istanbul ignore next */ []));
         this.canGoPrevious = computed(() => this.currentPage() > 1, /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "canGoPrevious" }] : /* istanbul ignore next */ []));
         this.isFirstPage = computed(() => this.currentPage() === 1, /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "isFirstPage" }] : /* istanbul ignore next */ []));
-        this.isLastPage = computed(() => this.currentPage() === this.totalPages(), /* @ts-ignore */
+        this.isLastPage = computed(() => this.currentPage() === this.effectiveTotalPages(), /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "isLastPage" }] : /* istanbul ignore next */ []));
         this.currentPageAnnouncement = computed(() => {
-            return `Page ${this.currentPage()} sur ${this.totalPages()}`;
+            return `Page ${this.currentPage()} sur ${this.effectiveTotalPages()}`;
         }, /* @ts-ignore */
         ...(ngDevMode ? [{ debugName: "currentPageAnnouncement" }] : /* istanbul ignore next */ []));
-        effect(() => {
-            const total = this.totalPages();
-            if (total < 1 || !Number.isFinite(total)) {
-                console.warn(`[psh-pagination] Invalid totalPages "${total}", setting to 1`);
-                this.totalPages.set(1);
-            }
-        });
+        if (isDevMode()) {
+            effect(() => {
+                // `0` is a legitimate state (empty list, or a server-side page count that is
+                // not loaded yet): stay silent. Only report what betrays a caller bug.
+                const total = this.totalPages();
+                if (total !== 0 && (!Number.isInteger(total) || total < 1)) {
+                    console.warn(`[psh-pagination] Invalid totalPages "${total}", using 1 page`);
+                }
+            });
+            effect(() => {
+                const maxVisible = this.maxVisiblePages();
+                if (maxVisible < 1 || !Number.isFinite(maxVisible)) {
+                    console.warn(`[psh-pagination] Invalid maxVisiblePages "${maxVisible}", must be >= 1`);
+                }
+            });
+        }
+        // `currentPage` IS owned by the component (it navigates), so correcting it with
+        // a write-back is the intended behaviour — clamped on the derived page count.
         effect(() => {
             const current = this.currentPage();
-            const total = this.totalPages();
+            const total = this.effectiveTotalPages();
             if (current < 1 || !Number.isFinite(current)) {
-                console.warn(`[psh-pagination] Invalid currentPage "${current}", setting to 1`);
+                if (isDevMode()) {
+                    console.warn(`[psh-pagination] Invalid currentPage "${current}", setting to 1`);
+                }
                 this.currentPage.set(1);
             }
-            else if (current > total && total >= 1) {
-                console.warn(`[psh-pagination] currentPage "${current}" exceeds totalPages "${total}", setting to ${total}`);
+            else if (current > total) {
+                if (isDevMode()) {
+                    console.warn(`[psh-pagination] currentPage "${current}" exceeds totalPages "${total}", setting to ${total}`);
+                }
                 this.currentPage.set(total);
             }
         });
-        effect(() => {
-            const maxVisible = this.maxVisiblePages();
-            if (maxVisible < 1 || !Number.isFinite(maxVisible)) {
-                console.warn(`[psh-pagination] Invalid maxVisiblePages "${maxVisible}", must be >= 1`);
-            }
-        });
     }
     getState() {
         if (this.currentPage() === 1)
             return 'first';
-        if (this.currentPage() === this.totalPages())
+        if (this.currentPage() === this.effectiveTotalPages())
             return 'last';
         return 'default';
     }
     goToPage(page) {
-        if (page !== this.currentPage() && page >= 1 && page <= this.totalPages()) {
+        const total = this.effectiveTotalPages();
+        if (page !== this.currentPage() && page >= 1 && page <= total) {
             this.currentPage.set(page);
             this.pageChange.emit(page);
         }
-        else if (page < 1 || page > this.totalPages()) {
+        else if (page < 1 || page > total) {
             this.navigationError.emit({
                 action: 'goToPage',
-                reason: `Page ${page} is out of bounds (1-${this.totalPages()})`,
+                reason: `Page ${page} is out of bounds (1-${total})`,
             });
         }
     }
@@ -3632,7 +3657,7 @@ class PshPaginationComponent {
         this.goToPage(1);
     }
     goToLastPage() {
-        this.goToPage(this.totalPages());
+        this.goToPage(this.effectiveTotalPages());
     }
     goToNextPage() {
         if (this.canGoNext()) {