keep highlighted track in focus / in sight when sorting tracks table

[Marco]

I've purchased a Denon MC7000 a few weeks ago (had a MC3000 for a long time before), so I was forced to upgrade from Mixxx 2.1.8 to Mixxx 2.2.4 to get it work.
Unfortunately I've noticed some weird behaviour when I click onto a track in library (it gets highlighted), and then try to sort it to BPM or Key to find a matching track. In Mixxx 2.1.8 this track was still visible in library after sorting, so I could quickly find a track which fits. Since Mixxx 2.2.4, the highlighted track gets out of focus / out of sight after sorting, so I have to scroll to find this highlighted track again, which is quite annoying and time consuming.
This bug still persists in 2.3.0 Beta (right now I'm testing commit 0a693b689a429582e3fa43ccce86b33465794e9f). Hope that someone can also confirm this.
OS: Gentoo Linux, Kernel 5.4.60

Kind regards,
Marco

[OsZ (toszlanyi)]

I can confirm this behavior as well. When sorting columns then the highlighted track is out of sight and need to be searched by scrolling through the library again.

[ronso0 (ronso0)]

Can you both confirm that in 2.1.8 (with identical track set) the track remained visible in the tracks table?

If so we could *try* to find the changed that causes this behaviour
(we're currently working on 2.3 & 2.4 and a lot changed since 2.1..)

[Marco (c-moon)]

Yes. confirm this, so I've made two screen capture videos (one with Mixxx 2.3.0 Beta, and one with the old Mixxx 2.1.8). Both are running on the same computer, so the environment is completely identical (e.g. Qt 5.14.2).

Mixxx 2.3.0 Beta:
-----------------
http://178.254.21.56/marco/mixxx/mixxx-230_beta.avi

Mixxx 2.1.8:
------------
http://178.254.21.56/marco/mixxx/mixxx-218_release.avi

HTH...

Marco

[ronso0 (ronso0)]

Code-wise I don't see which change from 2.1 > 2.2 > 2.3 could have caused this.
Seems to be an Qt issue.
This should be achieved by WTrackTableView::doSortByColumn but appearantly scrollTo() doesn't work.
https://github.com/mixxxdj/mixxx/blob/2.3/src/widget/wtracktableview.cpp#L855

working on it...

[ronso0 (ronso0)]

https://github.com/mixxxdj/mixxx/pull/3092

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/10091