Can no longer invert sort order when displayed as a tree

[cherio]

Version 3.3.0 (Arch Linux)

Test case:

1. Select tree view presentation (F5)
2. Make sure "Tree view is always sorted by PID" is not selected/unchecked in the settings
3. Choose a sort column, e.g. TIME (either with F6 or mouse)
4. Try to invert sort order with either pressing "I" or mouse click on the column header.

Expected behavior:

• The tree should get reverse-sorted
• The little up/down arrow in the header should reverse its direction

Observed behavior:

• The sort stays the same.
• The sort direction arrow only changes direction when using keyboard "I", but stays unchanged when using mouse.

[fasterit]

Cannot confirm, please try clicking the PID header ... it is most easy to see the inversion there.
TIME+ works here, too.

[BenBE]

The tree is sorted in a way, that the sort order is applied to the children (of the same level) of each parent …

Nicely visible with the RES (memory usage) column …

[cherio]

I've done some additional testing

• tried sorting by various columns including PID
• removed/re-installed htop from the standard Arch repo
• purged .config/htop
• updated the system
• I tried various terminals, including the basic tty

The sorting issue persisted after everything. I have the total of 3 Archlinux boxes and all of them have the issue.

At the same time I tested 3.3.0 in Ubuntu and Debian have no such issues.

Unless you have more insight, I guess this could be a dependency or Arch build issue

[cherio]

I ran 2 more tests, compiling it from scratch

1. I pulled the latest source from here, compiled it and ran. The issue was not present.
2. I pulled branch 3.3.0 and compiled it. The issue was present.

This means the issue is not Arch specific.

[fasterit]

That's why good distros backport patches :-)
https://sources.debian.org/patches/htop/3.3.0-4/

[eworm-de]

Ha, looks like we need an upstream release! 😜

Probably fixed by 91990b1...

[BenBE]

Ha, looks like we need an upstream release! 😜

Once the CPU topology stuff for libsensors are satisfactorily fixed …

[eworm-de]

The fix should be in htop 3.3.0-3. I guess we can close this now.

[fasterit]

cf. https://gitlab.archlinux.org/archlinux/packaging/packages/htop/-/commit/9ebbd551f16d3e50215c301d5c6a489306ffa764