Fixes for IE

diff --git a/ui-lib.pl b/ui-lib.pl
index 7531036..258b944 100644 (file)
--- a/ui-lib.pl
+++ b/ui-lib.pl
@@ -551,7 +551,6 @@ return $rv;
 # implemented using two <select> lists and Javascript buttons to move elements
 # between them. The resulting input value is \n separated.
 # XXX IE testing
-# XXX mobile theme
 sub ui_multi_select
 {
 return &theme_ui_multi_select(@_) if (defined(&theme_ui_multi_select));
@@ -610,13 +609,13 @@ var vals_idx = vals.selectedIndex;
 if (dir == 1 && opts_idx >= 0) {
        // Moving from options to selected list
        var o = opts.options[opts_idx];
-       vals.options[vals.options.length] = o;
+       vals.options[vals.options.length] = new Option(o.text, o.value);
        opts.remove(opts_idx);
        }
 else if (dir == 0 && vals_idx >= 0) {
        // Moving the other way
        var o = vals.options[vals_idx];
-       opts.options[opts.options.length] = o;
+       opts.options[opts.options.length] = new Option(o.text, o.value);
        vals.remove(vals_idx);
        }
 // Fill in hidden field

diff --git a/useradmin/edit_group.cgi b/useradmin/edit_group.cgi
index 4c2d180..9033efe 100755 (executable)
--- a/useradmin/edit_group.cgi
+++ b/useradmin/edit_group.cgi
@@ -71,6 +71,7 @@ print &ui_table_row(&hlink($text{'pass'}, "gpasswd"),
                       &ui_textbox("pass", undef, 15) ] ]));
 
 # Member chooser
+@ulist = &sort_users(\@ulist, $config{'sort_mode'});
 print &ui_table_row(&hlink($text{'gedit_members'}, "gmembers"),
        &ui_multi_select("members",
                [ map { [ $_, $_ ] } split(/,/ , $group{'members'}) ],