Page 1 of 1

2.53 beta 1: Can't sort Type column

Posted: 10 May 2010, 15:51
by JohnFredC
To reproduce:
  • Use "Choose columns" to add the "Type" column to a Defined View (I created a new View for this purpose).

    Return to the file panel and display that Defined View.

    Left-click the "Type" column label to sort the column.

    Salamander does not respond.
Of course a work-around that substitutes most of the time is to sort by EXT instead.

Re: 2.53 beta 1: Can't sort Type column

Posted: 10 May 2010, 19:04
by therube
Seems like a strange omission, or bug.

Oh, & this is not 2.53 specific as it looks to have been like this since at least 2.0.

Re: 2.53 beta 1: Can't sort Type column

Posted: 10 May 2010, 20:44
by Petr Solin
Yes, this is by design. You cannot sort by DOS Name column too.

Re: 2.53 beta 1: Can't sort Type column

Posted: 10 May 2010, 22:02
by JohnFredC
By design? That means intentional.

Why would you seek to prevent a user from sorting by the value of any column in the file list?

Re: 2.53 beta 1: Can't sort Type column

Posted: 10 May 2010, 22:07
by Petr Solin
We can add such feature in some future version, no problem. But at least for now it is not important thing, we need to do other things. ;-)

Re: 2.53 beta 1: Can't sort Type column

Posted: 10 May 2010, 22:18
by JohnFredC
Ah, OK. "By design" would definitely be the wrong phrase, then.

As a developer myself, however, it surprises me that the Salamander code doesn't call a generic column sort method that simply uses the contents of the column and is enabled for all columns.

You know: Columns.Sort(Direction) or Columns.Sort(SortMethod, Direction)

Is there something different about the DOS name and Type data that requires special handling?