DataTable: sort icons for ascending/descending swapped?

[StackedCrooked]

Describe the bug

I have a question about the default sorting icon used by the DataTable component, specifically this icon:

This icon is labeled pi-sort-amount-down-alt in the PrimeReact Icons documentation.

The arrow points from smallest to largest, which I think means ascending order. But in this example, it seems to mean descending:

I might be misinterpreting the symbolic meaning of the arrow and the lines. If that is the case, could you please clarify?

If it's a bug, could you please confirm?

Thanks!

Reproducer

https://stackblitz.com/edit/vitejs-vite-apq6rx?file=src%2FApp.tsx

PrimeReact version

18.2.0

React version

18.x

Language

TypeScript

Build / Runtime

Next.js

Browser(s)

No response

Steps to reproduce the behavior

1. Open the StackBlitz example.
2. Click on the "Name" column and observe the icons used to indicate ascending and descending order.

Expected behavior

I expect the icons for ascending and descending order to be swapped.

(Unless I am misinterpreting the symbols. If that is the case, I would appreciate an explanation.)

[melloware]

This has been raised before: primefaces/primevue#4708 Some people think its OK others think its backwards :)

[StackedCrooked]

This has been raised before: primefaces/primevue#4708 Some people think its OK others think its backwards :)

Thanks for the quick reply. I understand that it's subjective.

If I may share my perspective, I would prefer to stick to the convention used for years by the Windows and Mac operating systems, and use triangles like this:

Feel free to ignore my suggestion and proceed as you see fit. :)

Thanks again!

[melloware]

At my clients we have standardized on FontAwesome version:

[melloware]

Because these icons are used across 4 PrimTek libs you should really open this ticket at the PrimeIcons repo and not this project individually. as a workaround you can define your own icons in the table for sorting.

[onursenture]

Because these icons are used across 4 PrimTek libs you should really open this ticket at the PrimeIcons repo and not this project individually. as a workaround you can define your own icons in the table for sorting.

Actually, I asked him to open a ticket here because I think the PrimeVue version is the right one. https://github.com/orgs/primefaces/discussions/2276#discussioncomment-9983658

[melloware]

oh are we using the wrong icon @onursenture ? We are on PrimeIcons 7.0.0 already?

[onursenture]

It's not about the PrimeIcons version; as I remember, these are SVGs defined somewhere. The icons that we used on PrimeVue should be referenced.

---

[melloware]

Yep i will fix it.