cdda-itembrowser icon indicating copy to clipboard operation
cdda-itembrowser copied to clipboard

Sorting consumables by spoiling time not working properly

Open Nandorianen opened this issue 9 years ago • 2 comments

I think right now it sorts spoil tile as text "X days", which is why 5.00 > 41.25 > 4.00. All these values are given in days, so writing "days" everywhere is redundant. Moving it to the tooltip description of the column would allow sorting these values as numbers, which will result in proper sorting.

Nandorianen avatar Nov 08 '15 09:11 Nandorianen

I tweaked this, could you make sure it's now behaving the way you were expecting it to behave?

Sheco avatar Nov 10 '15 16:11 Sheco

Sure, it sorts as it should now. Thanks again!

Come to think of it, there are other columns with similar problems. "Weight" in armor, melee, ranged weapons, materials (in last case it's working as it should with current content, but future content may cause similar sorting issues). "Stim" in consumables. Seems to be it. At least everything else seems like proper numbers.

Nandorianen avatar Nov 10 '15 18:11 Nandorianen