Component: Paginator. "Show All" option in dropdown always displayed first

[markuczy]

Describe the bug

When you use paginator with rowsPerPageOptions property you can display "All" option. However, it will always be displayed first even if you define it as last in rowsPerPageOptions.

<p-paginator 
    ...
    [rowsPerPageOptions]="[10, 20, 30, { showAll: 'All'}]" />

I would expect that the following snippet would result in 'All' option displayed last in dropdown.
You could probably simply not do unshift but just push 'All' option whenever someone puts it in rowsPerPageOptions - https://github.com/primefaces/primeng/blob/master/src/app/components/paginator/paginator.ts#L409

Environment

stackblitz

Reproducer

https://stackblitz.com/edit/stackblitz-starters-t6faqdch?file=src%2Fapp%2Fapp.component.html

Angular version

^18.2.0

PrimeNG version

^18.0.0

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

stackblitz

Browser(s)

No response

Steps to reproduce the behavior

No response

Expected behavior

No response

[github-actions]

Due to PrimeNG team's busy roadmap, this issue is available for anyone to work on. Make sure to reference this issue in your pull request. ✨ Thank you for your contribution! ✨

[markuczy]

@mertsincan New implementation is not what I would expect either. What if I want to display the "ALL" button first? I cannot do that anymore...

Instead of pushing it as last argument, is it possible to simply push all items in the same order?

[markuczy]

That way users determine the order of the elements.

example 1: [rowsPerPageOptions]="[10, 20, 30, { showAll: 'All'}]" -> 'ALL' option displayed last
example 2: [rowsPerPageOptions]="[{ showAll: 'All'}, 10, 20, 30]" -> 'ALL' option displayed first
example 3: [rowsPerPageOptions]="[10, { showAll: 'All'}, 20, 30]" -> 'ALL' option displayed second

Example 3 is less likely to happen, however possible with easy change of implementation.