Re: Re: html_quickform_advmultiselect good news

On Monday 27 June 2005 03:19 pm, Jamie Alessio wrote: > Laurent, > Nice work on extending the advmultiselect code that I started and > turning it into an actual PEAR package! I hadn't been keeping up with > the pear-dev list and just today caught up on all the mail that was > exchanged about the development of HTML_QuickForm_advmultiselect. I > downloaded the package today via pear and tried it out in an application > I am working on and it appears to work great. > > Suggestions: > > *Sorting of select lists* > It would be nice to have an option for the select lists to sort > alphabetically when elements are moved between them. It is confusing > when the list starts out sorted and then when you move an option back > and forth between select lists it always gets added to the bottom. Of > course, not everything makes sense to be sorted alphabetically but I'd > guess that having that as the default option would work for the majority > of cases and people can disable it if they don't want the sorting (or > replace it with some other sorting routine that matches the data in the > select). > I think that it'd be better if it simply maintained the order of the options when they were added. i.e. add an 'order' var on each option in the original list (based on it's potision in the original markup), then sort based on that in the selected and unselected lists. It's simpler than your suggestion, and allows for the actual order to be determined on the backend, rather than shuffling things around when they're displayed.

Attachment: [application/pgp-signature]