-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Channels should be in alphabetical order #28
Comments
Assuming you mean the output of /list, channels are listed in the order that the IRCd provides them to Iris. If you want some sort of ordering, get your ircd to do it. |
I mean the channel switching tabs at the top. |
What if you had like #c1 #c2 #c3 #c10 #c20 ? How would you order that? User-order would be much nicer, IMO. |
You'd want a natural compare function here. pleanty exist |
You'd have to handle numbers, unicode, dates, and idk what else you can stick on channel names. Are you sure you want to do that? |
https://www.npmjs.org/package/natural-compare handles all of that (besides dates why would a channel name be a date?) |
"Works well with dates in ISO format" what about the hundreds of other date formats currently in use? Timestamps? (AM/PM vs 24h) |
@SoniEx2 There is a difference between "good enough" and "handles every edge case ever", in this case a simple alphabetical sort is "good enough". I don't care if its |
@SoniEx2 it's impossible to handle them all in any other place (think m/d/y and d/m/y in the same list), and who the hell puts dates in channel names anyway? so don't care about handling them all here, let's say it's just a bonus that the standard international date format sorts well. |
Well then at least make it configurable between user order (user can rearrange tabs) and ASCIIbetical/alphabetical/whatever order (default) |
No description provided.
The text was updated successfully, but these errors were encountered: