Cannot reorder the 'Name' column under Qt5

[sledgehammer999]

The title says it all.

I discovered this while testing the fix for #2340. I haven't dug into it. A soft(?) dependency for v3.2.0.

PS: This may need a fresh install to manifest.

[sledgehammer999]

@pmzqla can you confirm on your machine?

[pmzqla]

Yes, same here.

EDIT: It happens everywhere for the first column, even in "Trackers" where the first column is "#".
EDIT2: https://bugreports.qt.io/browse/QTBUG-33974

[sledgehammer999]

Supposedly QTreeView::setTreePosition with a value of -1 should fix this but I it doesn't.
Unless I don't understand the docs correctly.

Rant: Are they stupid? If setRootIsDecorated() is set to false, they should automatically allow the moving of the first column. Ideally the treestructure shouldn't be counted as a column at all. It should be part of the widget/view. Qt seems a very well though out API. I wonder how they didn't see this!?!?!

[pmzqla]

qt/qtbase@e0fc088
It seems it's just not possible. The hack in the commit message works though.

[sledgehammer999]

I chose not to switch to qt5(by default) for official builds due to this.

[davispuh]

Seems to be working fine with Qt 5.5.1

[sledgehammer999]

@davispuh Did you try to reorder your first column? Does it work?

[davispuh]

yeah

[pmzqla]

You can because that's not the actual first column. The first column is "#", which is hidden unless you enable torrent queuing from the settings. Try the same in the bottom panels, e.g. Tracker.

I quickly looked at the source code of Qt5.5 and the code that prevents movements of the first column is still there.

[davispuh]

indeed, can't move # but that's not big deal and Name is movable.

[ngosang]

This bug is going to be fixed in Qt? If not we can add a new column with size 0 in the first column or some hack.

[pmzqla]

It's done on purpose, so I don't think it's going to be "fixed". See the commit I linked here above for more info, it also shows a working hack.

[sledgehammer999]

@pmzqla IIRC trackers/peers/content columns cannot be moved anyway.

@davispuh All columns should be movable.

[ngosang]

@sledgehammer999 I think Qt developers are not going to allow reordering the first column. Last week they ported almost all applications included in the KDE desktop to Qt5 (around 150 apps) and in Dolphin (the default file explorer) and in other apps you can't move the first column.

[sledgehammer999]

@ngosang I saw your earlier post about the hack and was pondering over it while on vacation. Today I'll test if adding another dummy column first and hiding it solves the problem. If it does I'll commit it and close this.