cps-table/tree-table: changing rows-per-page on an empty table doesn't emit
Los mantenedores suelen responder en 1 día
@lukasmatta ya está trabajando en esto.
Desde el 5/11/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
When the table has no records (totalRecords = 0), picking a new value in the "Rows per page" select does nothing outside the table. onRowsPerPageChanged() sends the change through changePage(), which returns early because getPageCount() is 0. Neither pageChanged nor lazyLoaded fires. The select still shows the new value because rows is bound with ngModel.
In lazy mode, the consumer never learns about the new page size. When data arrives later (for example after a filter change), the request uses the old rows. The paginator then shows one page size while the rows come from a request using another.
Steps to reproduce:
- Render
cps-tablewith[lazy]="true",[paginator]="true"andtotalRecords = 0. - Change "Rows per page" from 10 to 25. Nothing is emitted.
- Change a filter in the consumer so that it fetches data.
- The consumer requests 10 rows while the select shows 25.
Expected: A change to the page size always emits pageChanged, and in lazy mode lazyLoaded, with first reset to 0, whatever the current record count is.
Suggested fix: In onRowsPerPageChanged, skip the page-count check. Call primengTable.onPageChange({ first, rows }) directly, or clamp the target page to max(pageCount - 1, 0). Add a spec covering an empty lazy table.
- Lenguaje dominante
- TypeScript
- Estrellas
- 9
- Forks
- 8
- Merge medio
- 1 d 1 h
- PR fusionados (30 d)
- 23
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de AbsaOSS/cps-shared-ui
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
AbsaOSS/cps-shared-ui#875 ·
Los mantenedores suelen responder en 1 día
-
InputComponent - expand type beyond text/number/passwordQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertoenhancement library
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
AbsaOSS/cps-shared-ui#644 ·
Los mantenedores suelen responder en 1 día
-
Esc: gaps in tooltip and notification dismissalPosiblemente ocupada @fateeand la tomó hace 1 día. Abiertoaccessibility bug library
AbsaOSS/cps-shared-ui#942 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
AbsaOSS/cps-shared-ui#940 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
AbsaOSS/cps-shared-ui#939 ·
Los mantenedores suelen responder en 1 día
Todos los issues de AbsaOSS/cps-shared-ui
Issues similares
-
Cleanup SetupPosiblemente ocupada @AlexanderKaran la tomó hoy. Abiertodocumentation enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
e18e/framework-tracker#769 ·
Los mantenedores suelen responder en 1 día
-
fix(docker): correct stale --db-path comments in fetch/git Dockerfiles and clarify git READMEAbiertobug v2
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
modelcontextprotocol/servers#5483 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
XiaomiMiMo/MiMo-Code#2783 ·
Los mantenedores suelen responder en 1 día
-
area: web bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
kinqsradio/orbyn-engineering#227 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
joeldesante/TurfBuilder#258 ·
Los mantenedores suelen responder en 1 día