omemo-top icon indicating copy to clipboard operation
omemo-top copied to clipboard

Actually sort the list

Open htgoebel opened this issue 5 years ago • 6 comments

While there are several closed issues about the list to be sorted (#21, #38), the list still is not sorted. Please sort it. Thanks.

Since sorting might not be that easy in liquid, https://stackoverflow.com/questions/8991995/using-liquid-to-sort-posts-alphabetically might be of help.

htgoebel avatar Jan 10 '19 22:01 htgoebel

@htgoebel The list is initially sorted by implementation status and can be sorted by any of the available columns: screenshot-2019-01-11-094935-811x122

Sorting relies on JavaScript to be enabled though.

bascht avatar Jan 11 '19 08:01 bascht

Sorting relies on JavaScript to be enabled though.

The list should be sorted per default and not rely on Javascript. For me it was not even sorted when enabling Javascript.

htgoebel avatar Jan 11 '19 09:01 htgoebel

@htgoebel You might be blocking our CDN then – could you check your Browser Console for JavaScript errors?

bascht avatar Jan 14 '19 08:01 bascht

My point is: The list should already be sorted in the html delivered and not rely on being sorted by Javascript on the client side.

htgoebel avatar Jan 14 '19 08:01 htgoebel

@htgoebel I think you made that point somewhat clear, I was just asking whether you saw any JavaScript errors. Pre-Sorting the list before rendering the HTML is something we could add, but then it'd be only sorted by e.g. status or name.

bascht avatar Jan 14 '19 08:01 bascht

IC. I can't reproduce the "not even sorted when enabling Javascript" when I tried again just now. Sorry, this part looks like a false alarm.

htgoebel avatar Jan 14 '19 09:01 htgoebel