[Performance issue] Filament table by default have list all records enabled

[no-operation]

The method below has “all” as the default option, which can cause problems if the number of records grows significantly. It’s not a good practice to have “all” as the default. I suggest using only fixed values by default, and the developer can add the “all” option if they specifically need it.

/**
 * @return array<int | string>
 */
public function getPaginationPageOptions(): array
{
    return $this->evaluate($this->paginationPageOptions) ?? [5, 10, 25, 50, 'all'];
}

It should be changed to:

/**
 * @return array<int | string>
 */
public function getPaginationPageOptions(): array
{
    return $this->evaluate($this->paginationPageOptions) ?? [5, 10, 25, 50];
}

What do you think?