fix #15024 table filter row aria-labels

[TaneliTuomola]

fixes #15024
Added aria-label support.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
primeng	⬜️ Ignored (Inspect)	Visit Preview		Mar 13, 2024 7:44am

[cetincakiroglu]

Hi,

Thanks for the effort and support, your fix is correct but we need to get ariaLabel from the translateService to make it compatible with i18n. Could you please update the PR?

It should look like this:

// taken from the table.ts line 5283
    get showMenuButtonLabel() {
        return this.config.getTranslation(TranslationKeys.SHOW_FILTER_MENU);
    }

Please tag me after updating the PR.

[TaneliTuomola]

I thought that it would be better to be able to set aria-label to each p-columnFilter element separately (via input) and let the client application handle translations if needed. In your suggestion there will be only one generic aria-label in every p-columnFilter element in every table? @cetincakiroglu