Skip to content
This repository

Multiselect: sort checked up top #971

Closed
jancborchardt opened this Issue December 20, 2012 · 7 comments

2 participants

Jan-Christoph Borchardt Thomas Tanghus
Jan-Christoph Borchardt
Collaborator

In users for example, the multiselect plugin should automatically sort the selected entries to be first because they are the most relevant.

cc @tanghus @icewind1991

Thomas Tanghus
Collaborator

I could add an option 'sorting':'alphaNumeric'|'checkedFirst'|'none' where 'none' is default.

Jan-Christoph Borchardt
Collaborator

@tanghus let’s just do it properly, not with an option. People shouldn’t be able to configure it to behave shittily. ;)

Thomas Tanghus
Collaborator

@jancborchardt I'd rather say let's do it properly, with an option. Programmers don't want shitty defaults ;) People - as in users - won't be able to configure it.

Jan-Christoph Borchardt
Collaborator

Developers are users too. And if there’s the option it will be changed, which will turn out crappy for users. Let’s see the options:

  • »none«: what does that exactly mean? Isn’t that the same as alphanumeric? If you mean sort by order when it was added: makes no sense when you come back to it later.
  • »alphaNumeric« is standard
  • »checkedFirst« would also be alphanumeric, just with the checked (read:relevant) entries sorted up top

Of course options always seem cool, but most of the time they just don’t make sense.

Thomas Tanghus
Collaborator

Well "none" shouldn't be an option but what happened when the sorting argument wasn't given - as it is now.

It's not because options are cool, but because it would be a bad design decision to rearrange the list per default. The plugin shouldn't be tailored just to work in one use case, then it's not a plugin.

Thomas Tanghus
Collaborator
Jan-Christoph Borchardt
Collaborator

Since #994 is merged in this is fixed, yeah!

Jan-Christoph Borchardt jancborchardt closed this January 12, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.