Handle hostnames with upper-case letters
[webmin.git] / ui-lib.pl
old mode 100644 (file)
new mode 100755 (executable)
index 6f97244..178677f
--- a/ui-lib.pl
+++ b/ui-lib.pl
@@ -1,9 +1,13 @@
+use vars qw($theme_no_table $ui_radio_selector_donejs $module_name 
+           $ui_multi_select_donejs);
+
 =head1 ui-lib.pl
 
 Common functions for generating HTML for Webmin user interface elements.
 Some example code :
 
- require '../ui-lib.pl';
+ use WebminCore;
+ init_config();
  ui_print_header(undef, 'My Module', '');
 
  print ui_form_start('save.cgi');
@@ -260,7 +264,7 @@ $rv .= "</tr>\n";
 return $rv;
 }
 
-=head2 ui_checked_columns_row(&columns, &tdtags, checkname, checkvalue, [checked?], [disabled])
+=head2 ui_checked_columns_row(&columns, &tdtags, checkname, checkvalue, [checked?], [disabled], [tags])
 
 Returns HTML for a row in a multi-column table, in which the first column 
 contains a checkbox. The parameters are :
@@ -277,15 +281,17 @@ contains a checkbox. The parameters are :
 
 =item disabled - Set to 1 if the checkbox should be disabled and thus un-clickable.
 
+=item tags - Extra HTML tags to include in the radio button.
+
 =cut
 sub ui_checked_columns_row
 {
 return &theme_ui_checked_columns_row(@_) if (defined(&theme_ui_checked_columns_row));
-my ($cols, $tdtags, $checkname, $checkvalue, $checked, $disabled) = @_;
+my ($cols, $tdtags, $checkname, $checkvalue, $checked, $disabled, $tags) = @_;
 my $rv;
 $rv .= "<tr $cb class='ui_checked_columns'>\n";
 $rv .= "<td class='ui_checked_checkbox' ".$tdtags->[0].">".
-       &ui_checkbox($checkname, $checkvalue, undef, $checked, undef, $disabled).
+       &ui_checkbox($checkname, $checkvalue, undef, $checked, $tags, $disabled).
        "</td>\n";
 my $i;
 for($i=0; $i<@$cols; $i++) {
@@ -304,7 +310,7 @@ $rv .= "</tr>\n";
 return $rv;
 }
 
-=head2 ui_radio_columns_row(&columns, &tdtags, checkname, checkvalue, [checked], [disabled])
+=head2 ui_radio_columns_row(&columns, &tdtags, checkname, checkvalue, [checked], [disabled], [tags])
 
 Returns HTML for a row in a multi-column table, in which the first
 column is a radio button. The parameters are :
@@ -321,11 +327,13 @@ column is a radio button. The parameters are :
 
 =item disabled - Set to 1 if the radio button should be disabled and thus un-clickable.
 
+=item tags - Extra HTML tags to include in the radio button.
+
 =cut
 sub ui_radio_columns_row
 {
 return &theme_ui_radio_columns_row(@_) if (defined(&theme_ui_radio_columns_row));
-my ($cols, $tdtags, $checkname, $checkvalue, $checked, $dis) = @_;
+my ($cols, $tdtags, $checkname, $checkvalue, $checked, $dis, $tags) = @_;
 my $rv;
 $rv .= "<tr $cb class='ui_radio_columns'>\n";
 $rv .= "<td class='ui_radio_radio' ".$tdtags->[0].">".
@@ -437,13 +445,15 @@ foreach my $r (@$data) {
                        # Checkbox in non-first column
                        push(@cols, &ui_checkbox($c->{'name'}, $c->{'value'},
                                                 $c->{'label'}, $c->{'checked'},
-                                                undef, $c->{'disabled'}));
+                                                $c->{'tags'},
+                                                $c->{'disabled'}));
                        }
                elsif ($c->{'type'} eq 'radio') {
                        # Radio button in non-first column
                        push(@cols, &ui_oneradio($c->{'name'}, $c->{'value'},
                                                 $c->{'label'}, $c->{'checked'},
-                                                undef, $c->{'disabled'}));
+                                                $c->{'tags'},
+                                                $c->{'disabled'}));
                        }
                elsif ($c->{'type'} eq 'group') {
                        # Header row that spans whole table
@@ -471,12 +481,14 @@ foreach my $r (@$data) {
        elsif ($c0->{'type'} eq 'checkbox') {
                $rv .= &ui_checked_columns_row(\@cols, \@rtds, $c0->{'name'},
                                               $c0->{'value'}, $c0->{'checked'},
-                                              $c0->{'disabled'});
+                                              $c0->{'disabled'},
+                                              $c0->{'tags'});
                }
        elsif ($c0->{'type'} eq 'radio') {
                $rv .= &ui_radio_columns_row(\@cols, \@rtds, $c0->{'name'},
                                             $c0->{'value'}, $c0->{'checked'},
-                                            $c0->{'disabled'});
+                                            $c0->{'disabled'},
+                                            $c0->{'tags'});
                }
        }
 
@@ -484,7 +496,7 @@ $rv .= &ui_columns_end();
 return $rv;
 }
 
-=head2 ui_form_columns_table(cgi, &buttons, select-all, &otherlinks, &hiddens, &headings, width-percent, &data, &types, no-sort, title, empty-msg)
+=head2 ui_form_columns_table(cgi, &buttons, select-all, &otherlinks, &hiddens, &headings, width-percent, &data, &types, no-sort, title, empty-msg, form-no)
 
 Similar to ui_columns_table, but wrapped in a form. Parameters are :
 
@@ -506,7 +518,7 @@ sub ui_form_columns_table
 return &theme_ui_form_columns_table(@_)
        if (defined(&theme_ui_form_columns_table));
 my ($cgi, $buttons, $selectall, $others, $hiddens,
-       $heads, $width, $data, $types, $nosort, $title, $emptymsg) = @_;
+       $heads, $width, $data, $types, $nosort, $title, $emptymsg, $formno) = @_;
 my $rv;
 
 # Build links
@@ -529,8 +541,8 @@ if (@$data) {
                                }
                        }
                if ($cbname) {
-                       unshift(@leftlinks, &select_all_link($cbname),
-                                           &select_invert_link($cbname));
+                       unshift(@leftlinks, &select_all_link($cbname, $formno),
+                                   &select_invert_link($cbname, $formno));
                        }
                }
        }
@@ -704,12 +716,19 @@ options. May truncate values to 2 decimal points! The parameters are :
 
 =item disabled - Set to 1 if this text box should be disabled by default.
 
+=item tags - Additional HTML attributes for the <input> tag.
+
+=item defaultunits - Units mode selected by default
+
 =cut
 sub ui_bytesbox
 {
-my ($name, $bytes, $size, $dis) = @_;
+my ($name, $bytes, $size, $dis, $tags, $defaultunits) = @_;
 my $units = 1;
-if ($bytes >= 10*1024*1024*1024*1024) {
+if ($bytes eq '' && $defaultunits) {
+       $units = $defaultunits;
+       }
+elsif ($bytes >= 10*1024*1024*1024*1024) {
        $units = 1024*1024*1024*1024;
        }
 elsif ($bytes >= 10*1024*1024*1024) {
@@ -729,7 +748,7 @@ if ($bytes ne "") {
        $bytes =~ s/\.00$//;
        }
 $size = &ui_max_text_width($size || 8);
-return &ui_textbox($name, $bytes, $size, $dis)." ".
+return &ui_textbox($name, $bytes, $size, $dis, undef, $tags)." ".
        &ui_select($name."_units", $units,
                 [ [ 1, "bytes" ],
                   [ 1024, "kB" ],
@@ -763,7 +782,7 @@ return "<input class='ui_upload' type=file name=\"".&quote_escape($name)."\" ".
        ($tags ? " ".$tags : "").">";
 }
 
-=head2 ui_password(name, value, size, [disabled?], [maxlength])
+=head2 ui_password(name, value, size, [disabled?], [maxlength], [tags])
 
 Returns HTML for a password text input. Parameters are the same as ui_textbox,
 and behaviour is identical except that the user's input is not visible.
@@ -772,13 +791,14 @@ and behaviour is identical except that the user's input is not visible.
 sub ui_password
 {
 return &theme_ui_password(@_) if (defined(&theme_ui_password));
-my ($name, $value, $size, $dis, $max) = @_;
+my ($name, $value, $size, $dis, $max, $tags) = @_;
 $size = &ui_max_text_width($size);
 return "<input class='ui_password' ".
        "type=password name=\"".&quote_escape($name)."\" ".
        "value=\"".&quote_escape($value)."\" ".
        "size=$size ".($dis ? "disabled=true" : "").
        ($max ? " maxlength=$max" : "").
+       " ".$tags.
        ">";
 }
 
@@ -901,13 +921,13 @@ if (defined($opts_title)) {
        }
 $rv .= "<tr class='ui_multi_select_row'>";
 $rv .= "<td>".&ui_select($name."_opts", [ ], $leftover,
-                        $size, 0, 0, $dis, $wstyle)."</td>\n";
-$rv .= "<td>".&ui_button("->", undef, $dis,
+                        $size, 1, 0, $dis, $wstyle)."</td>\n";
+$rv .= "<td>".&ui_button("->", $name."_add", $dis,
                 "onClick='multi_select_move(\"$name\", form, 1)'")."<br>".
-             &ui_button("<-", undef, $dis,
+             &ui_button("<-", $name."_remove", $dis,
                 "onClick='multi_select_move(\"$name\", form, 0)'")."</td>\n";
 $rv .= "<td>".&ui_select($name."_vals", [ ], $values,
-                        $size, 0, 0, $dis, $wstyle)."</td>\n";
+                        $size, 1, 0, $dis, $wstyle)."</td>\n";
 $rv .= "</tr></table>\n";
 $rv .= &ui_hidden($name, join("\n", map { $_->[0] } @$values));
 return $rv;
@@ -933,15 +953,27 @@ var opts_idx = opts.selectedIndex;
 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] = new Option(o.text, o.value);
-       opts.remove(opts_idx);
+       for(var i=0; i<opts.options.length; i++) {
+               var o = opts.options[i];
+               if (o.selected) {
+                       vals.options[vals.options.length] =
+                               new Option(o.text, o.value);
+                       opts.remove(i);
+                       i--;
+                       }
+               }
        }
 else if (dir == 0 && vals_idx >= 0) {
        // Moving the other way
-       var o = vals.options[vals_idx];
-       opts.options[opts.options.length] = new Option(o.text, o.value);
-       vals.remove(vals_idx);
+       for(var i=0; i<vals.options.length; i++) {
+               var o = vals.options[i];
+               if (o.selected) {
+                       opts.options[opts.options.length] =
+                               new Option(o.text, o.value);
+                       vals.remove(i);
+                       i--;
+                       }
+               }
        }
 // Fill in hidden field
 var hid = f.elements[name];
@@ -1174,11 +1206,13 @@ a field with radio buttons next to it. The parameters are :
 
 =item max - Optional maximum allowed input length, in characters.
 
+=item tags - Additional HTML attributes for the text box
+
 =cut
 sub ui_opt_textbox
 {
 return &theme_ui_opt_textbox(@_) if (defined(&theme_ui_opt_textbox));
-my ($name, $value, $size, $opt1, $opt2, $dis, $extra, $max) = @_;
+my ($name, $value, $size, $opt1, $opt2, $dis, $extra, $max, $tags) = @_;
 my $dis1 = &js_disable_inputs([ $name, @$extra ], [ ]);
 my $dis2 = &js_disable_inputs([ ], [ $name, @$extra ]);
 my $rv;
@@ -1189,7 +1223,9 @@ $rv .= &ui_radio($name."_def", $value eq '' ? 1 : 0,
 $rv .= "<input class='ui_opt_textbox' name=\"".&quote_escape($name)."\" ".
        "size=$size value=\"".&quote_escape($value)."\" ".
        ($value eq "" || $dis ? "disabled=true" : "").
-       ($max ? " maxlength=$max" : "").">\n";
+       ($max ? " maxlength=$max" : "").
+       " ".$tags.
+       ">\n";
 return $rv;
 }
 
@@ -1265,7 +1301,21 @@ return "<input type=button".
 =head2 ui_date_input(day, month, year, day-name, month-name, year-name, [disabled?])
 
 Returns HTML for a date-selection field, with day, month and year inputs.
-XXX
+The parameters are :
+
+=item day - Initial day of the month.
+
+=item month - Initial month of the year, indexed from 1.
+
+=item year - Initial year, four-digit.
+
+=item day-name - Name of the day input field.
+
+=item month-name - Name of the month select field.
+
+=item year-name - Name of the year input field.
+
+=item disabled - Set to 1 to disable all fields by default.
 
 =cut
 sub ui_date_input
@@ -1284,12 +1334,30 @@ $rv .= "</span>";
 return $rv;
 }
 
+=head2 ui_buttons_start
+
+Returns HTML for the start of a block of action buttoms with descriptions, as
+generated by ui_buttons_row. Some example code :
+
+  print ui_buttons_start();
+  print ui_buttons_row('start.cgi', 'Start server',
+                       'Click this button to start the server process');
+  print ui_buttons_row('stop.cgi', 'Stop server',
+                       'Click this button to stop the server process');
+  print ui_buttons_end();
+
+=cut
 sub ui_buttons_start
 {
 return &theme_ui_buttons_start(@_) if (defined(&theme_ui_buttons_start));
 return "<table width=100% class='ui_buttons_table'>\n";
 }
 
+=head2 ui_buttons_end
+
+Returns HTML for the end of a block started by ui_buttons_start.
+
+=cut
 sub ui_buttons_end
 {
 return &theme_ui_buttons_end(@_) if (defined(&theme_ui_buttons_end));
@@ -1298,7 +1366,20 @@ return "</table>\n";
 
 =head2 ui_buttons_row(script, button-label, description, [hiddens], [after-submit], [before-submit]) 
 
-MISSING DOCUMENTATION
+Returns HTML for a button with a description next to it, and perhaps other
+inputs. The parameters are :
+
+=item script - CGI script that this button submits to, like start.cgi.
+
+=item button-label - Text to appear on the button.
+
+=item description - Text to appear next to the button, describing in more detail what it does.
+
+=item hiddens - HTML for hidden fields to include in the form this function generates.
+
+=item after-submit - HTML for text or inputs to appear after the submit button.
+
+=item before-submit - HTML for text or inputs to appear before the submit button.
 
 =cut
 sub ui_buttons_row
@@ -1318,7 +1399,7 @@ return "<form action=$script class='ui_buttons_form'>\n".
 
 =head2 ui_buttons_hr([title])
 
-MISSING DOCUMENTATION
+Returns HTML for a separator row, for use inside a ui_buttons_start block.
 
 =cut
 sub ui_buttons_hr
@@ -1337,7 +1418,9 @@ else {
 
 =head2 ui_post_header([subtext])
 
-Returns HTML to appear directly after a standard header() call
+Returns HTML to appear directly after a standard header() call. This is never
+called directly - instead, ui_print_header calls it. But it can be overridden
+by themes.
 
 =cut
 sub ui_post_header
@@ -1354,7 +1437,9 @@ return $rv;
 
 =head2 ui_pre_footer
 
-Returns HTML to appear directly before a standard footer() call
+Returns HTML to appear directly before a standard footer() call. This is never
+called directly - instead, ui_print_footer calls it. But it can be overridden
+by themes.
 
 =cut
 sub ui_pre_footer
@@ -1367,10 +1452,35 @@ if (!$tconfig{'nohr'} && !$tconfig{'nobottomhr'}) {
 return $rv;
 }
 
-=head2 ui_print_header(subtext, args...)
+=head2 ui_print_header(subtext, image, [help], [config], [nomodule], [nowebmin], [rightside], [head-stuff], [body-stuff], [below])
 
 Print HTML for a header with the post-header line. The args are the same
-as those passed to header()
+as those passed to header(), defined in web-lib-funcs.pl, with the addition
+of the subtext parameter :
+
+=item subtext - Text to display below the title
+
+=item title - The text to show at the top of the page
+
+=item image - An image to show instead of the title text. This is typically left blank.
+
+=item help - If set, this is the name of a help page that will be linked to in the title.
+
+=item config - If set to 1, the title will contain a link to the module's config page.
+
+=item nomodule - If set to 1, there will be no link in the title section to the module's index.
+
+=item nowebmin - If set to 1, there will be no link in the title section to the Webmin index.
+
+=item rightside - HTML to be shown on the right-hand side of the title. Can contain multiple lines, separated by <br>. Typically this is used for links to stop, start or restart servers.
+
+=item head-stuff - HTML to be included in the <head> section of the page.
+
+=item body-stuff - HTML attributes to be include in the <body> tag.
+
+=item below - HTML to be displayed below the title. Typically this is used for application or server version information.
+
+
 
 =cut
 sub ui_print_header
@@ -1385,7 +1495,8 @@ print &ui_post_header($text);
 =head2 ui_print_unbuffered_header(subtext, args...)
 
 Like ui_print_header, but ensures that output for this page is not buffered
-or contained in a table.
+or contained in a table. This should be called by scripts that are producing
+output while performing some long-running process.
 
 =cut
 sub ui_print_unbuffered_header
@@ -1400,7 +1511,7 @@ $theme_no_table = 1;
 =head2 ui_print_footer(args...)
 
 Print HTML for a footer with the pre-footer line. Args are the same as those
-passed to footer()
+passed to footer().
 
 =cut
 sub ui_print_footer
@@ -1414,7 +1525,7 @@ print &ui_pre_footer();
 =head2 ui_config_link(text, &subs)
 
 Returns HTML for a module config link. The first non-null sub will be
-replaced with the appropriate URL.
+replaced with the appropriate URL for the module's config page.
 
 =cut
 sub ui_config_link
@@ -1444,7 +1555,7 @@ exit;
 
 =head2 ui_subheading(text, ...)
 
-Returns HTML for a section heading
+Returns HTML for a section heading whose message is the given text strings.
 
 =cut
 sub ui_subheading
@@ -1456,6 +1567,9 @@ return "<h3 class='ui_subheading'>".join("", @_)."</h3>\n";
 =head2 ui_links_row(&links)
 
 Returns HTML for a row of links, like select all / invert selection / add..
+Each element of the links array ref should be an HTML fragment like :
+
+  <a href='user_form.cgi'>Create new user</a>
 
 =cut
 sub ui_links_row
@@ -1470,7 +1584,8 @@ return @$links ? join("\n|\n", @$links)."<br>\n"
 
 =head2 ui_hidden_javascript
 
-Returns <script> and <style> sections for hiding functions and CSS
+Returns <script> and <style> sections for hiding functions and CSS. For
+internal use only.
 
 =cut
 sub ui_hidden_javascript
@@ -1551,7 +1666,16 @@ EOF
 =head2 ui_hidden_start(title, name, status, thisurl)
 
 Returns HTML for the start of a collapsible hidden section, such as for
-advanced options.
+advanced options. When clicked on, the section header will expand to display
+whatever is between this function and ui_hidden_end. The parameters are :
+
+=item title - Text for the start of this hidden section.
+
+=item name - A unique name for this section.
+
+=item status - 1 if it should be initially open, 0 if not.
+
+=item thisurl - URL of the current page. This is used by themes on devices that don't support Javascript to implement the opening and closing.
 
 =cut
 sub ui_hidden_start
@@ -1574,7 +1698,7 @@ return $rv;
 
 =head2 ui_hidden_end(name)
 
-Returns HTML for the end of a hidden section
+Returns HTML for the end of a hidden section, started by ui_hidden_start.
 
 =cut
 sub ui_hidden_end
@@ -1587,7 +1711,8 @@ return "</div>\n";
 =head2 ui_hidden_table_row_start(title, name, status, thisurl)
 
 Similar to ui_hidden_start, but for use within a table started with
-ui_table_start
+ui_table_start. I recommend against using this where possible, as it can
+be difficult for some themes to implement.
 
 =cut
 sub ui_hidden_table_row_start
@@ -1614,7 +1739,7 @@ return $rv;
 
 =head2 ui_hidden_table_row_end(name)
 
-MISSING DOCUMENTATION
+Returns HTML to end a block started by ui_hidden_table_start.
 
 =cut
 sub ui_hidden_table_row_end
@@ -1627,7 +1752,25 @@ return "</table></div><table width=100%>\n";
 
 =head2 ui_hidden_table_start(heading, [tabletags], [cols], name, status, [&default-tds], [rightheading])
 
-A table with a heading and table inside, and which is collapsible
+Returns HTML for the start of a form block into which labelled inputs can
+be placed, which is collapsible by clicking on the header. Basically the same
+as ui_table_start, and must contain HTML generated by ui_table_row.
+
+The parameters are :
+
+=item heading - Text to show at the top of the form.
+
+=item tabletags - HTML attributes to put in the outer <table>, typically something like width=100%.
+
+=item cols - Desired number of columns for labels and fields. Defaults to 4, but can be 2 for forms with lots of wide inputs.
+
+=item name - A unique name for this table.
+
+=item status - Set to 1 if initially open, 0 if initially closed.
+
+=item default-tds - An optional array reference of HTML attributes for the <td> tags in each row of the table.
+
+=item right-heading - HTML to appear in the heading, aligned to the right.
 
 =cut
 sub ui_hidden_table_start
@@ -1667,8 +1810,8 @@ return $rv;
 
 =head2 ui_hidden_table_end(name)
 
-Returns HTML for the end of table with hiding, as started by
-ui_hidden_table_start
+Returns HTML for the end of a form block with hiding, as started by
+ui_hidden_table_start.
 
 =cut
 sub ui_hidden_table_end
@@ -1680,8 +1823,32 @@ return "</table></div></td></tr></table>\n";
 
 =head2 ui_tabs_start(&tabs, name, selected, show-border)
 
-Render a row of tabs from which one can be selected. Each tab is an array
-ref containing a name, title and link.
+Returns a row of tabs from which one can be selected, displaying HTML
+associated with that tab. The parameters are :
+
+=item tabs - An array reference of array refs, each of which contains the value and user-visible text for a tab.
+
+=item name - Name of the HTML field into which the selected tab will be placed.
+
+=item selected - Value for the tab selected by default.
+
+=item show-border - Set to 1 if there should be a border around the contents of the tabs.
+
+Example code :
+
+  @tabs = ( [ 'list', 'List services' ],
+            [ 'install', 'Install new service' ] );
+  print ui_tabs_start(\@tabs, 'mode', 'list');
+
+  print ui_tabs_start_tab('mode', 'list');
+  generate_service_list();
+  print ui_tabs_end_tab('mode', 'list');
+
+  print ui_tabs_start_tab('mode', 'install');
+  generate_install_form();
+  print ui_tabs_end_tab('mode', 'install);
+
+  print ui_tabs_end();
 
 =cut
 sub ui_tabs_start
@@ -1694,8 +1861,8 @@ if (!$main::ui_hidden_start_donejs++) {
        }
 
 # Build list of tab titles and names
-my $tabnames = "[".join(",", map { "\"".&html_escape($_->[0])."\"" } @$tabs)."]";
-my $tabtitles = "[".join(",", map { "\"".&html_escape($_->[1])."\"" } @$tabs)."]";
+my $tabnames = "[".join(",", map { "\"".&quote_escape($_->[0])."\"" } @$tabs)."]";
+my $tabtitles = "[".join(",", map { "\"".&quote_escape($_->[1])."\"" } @$tabs)."]";
 $rv .= "<script>\n";
 $rv .= "document.${name}_tabnames = $tabnames;\n";
 $rv .= "document.${name}_tabtitles = $tabtitles;\n";
@@ -1714,7 +1881,7 @@ $rv .= "</td></tr>\n";
 $rv .= "<tr>\n";
 $rv .= "<td bgcolor=#ffffff width=1><img src=$imgdir/1x1.gif></td>\n";
 foreach my $t (@$tabs) {
-       if ($t ne $tabs[0]) {
+       if ($t ne $$tabs[0]) {
                # Spacer
                $rv .= "<td width=2 bgcolor=#ffffff class='ui_tab_spacer'>".
                       "<img src=$imgdir/1x1.gif></td>\n";
@@ -1763,9 +1930,10 @@ $main::ui_tabs_selected = $sel;
 return $rv;
 }
 
-=head2 ui_tabs_end(border)
+=head2 ui_tabs_end(show-border)
 
-MISSING DOCUMENTATION
+Returns HTML to end a block started by ui_tabs_start. The show-border parameter
+must match the parameter with the same name in the start function.
 
 =cut
 sub ui_tabs_end
@@ -1786,7 +1954,8 @@ return $rv;
 
 =head2 ui_tabs_start_tab(name, tab)
 
-Must be called before outputting the HTML for the named tab
+Must be called before outputting the HTML for the named tab, and returns HTML
+for the required <div> block. 
 
 =cut
 sub ui_tabs_start_tab
@@ -1801,7 +1970,9 @@ return $rv;
 
 =head2 ui_tabs_start_tabletab(name, tab)
 
-Behaves like ui_tabs_start_tab, but for use within a ui_table_start block
+Behaves like ui_tabs_start_tab, but for use within a ui_table_start block. 
+I recommend against using this where possible, as it is difficult for themes
+to implement.
 
 =cut
 sub ui_tabs_start_tabletab
@@ -1812,12 +1983,22 @@ my $div = &ui_tabs_start_tab(@_);
 return "</table>\n".$div."<table width=100%>\n";
 }
 
+=head2 ui_tabs_end_tab
+
+Returns HTML for the end of a block started by ui_tabs_start_tab.
+
+=cut
 sub ui_tabs_end_tab
 {
 return &theme_ui_tabs_end_tab(@_) if (defined(&theme_ui_tabs_end_tab));
 return "</div>\n";
 }
 
+=head2 ui_tabs_end_tabletab
+
+Returns HTML for the end of a block started by ui_tabs_start_tabletab.
+
+=cut
 sub ui_tabs_end_tabletab
 {
 return &theme_ui_tabs_end_tabletab(@_)
@@ -1827,7 +2008,8 @@ return "</table></div><table width=100%>\n";
 
 =head2 ui_max_text_width(width, [text-area?])
 
-Returns a new width for a text field, based on theme settings
+Returns a new width for a text field, based on theme settings. For internal
+use only.
 
 =cut
 sub ui_max_text_width
@@ -1842,8 +2024,13 @@ return $max && $w > $max ? $max : $w;
 =head2 ui_radio_selector(&opts, name, selected)
 
 Returns HTML for a set of radio buttons, each of which shows a different
-block of HTML when selected. &opts is an array ref to arrays containing
-[ value, label, html ]
+block of HTML when selected. The parameters are :
+
+=item opts - An array ref to arrays containing [ value, label, html ]
+
+=item name - HTML name for the radio buttons
+
+=item selected - Value for the initially selected button.
 
 =cut
 sub ui_radio_selector
@@ -1890,11 +2077,23 @@ EOF
 
 ####################### grid layout functions
 
-=head2 ui_grid_table(&elements, columns, [width-percent], [tds], [tabletags], [title])
+=head2 ui_grid_table(&elements, columns, [width-percent], [&tds], [tabletags], [title])
 
 Given a list of HTML elements, formats them into a table with the given
 number of columns. However, themes are free to override this to use fewer
-columns where space is limited.
+columns where space is limited. Parameters are :
+
+=item elements - An array reference of table elements, each of which can be any HTML you like.
+
+=item columns - Desired number of columns in the grid.
+
+=item width-percent - Optional desired width as a percentage.
+
+=item tds - Array ref of HTML attributes for <td> tags in the tables.
+
+=item tabletags - HTML attributes for the <table> tag.
+
+=item title - Optional title to add to the top of the grid.
 
 =cut
 sub ui_grid_table
@@ -1932,24 +2131,33 @@ if (defined($title)) {
 return $rv;
 }
 
-=head2 ui_radio_table(name, selected, &rows)
+=head2 ui_radio_table(name, selected, &rows, [no-bold])
 
 Returns HTML for a table of radio buttons, each of which has a label and
-some associated inputs to the right.
+some associated inputs to the right. The parameters are :
+
+=item name - Unique name for this table, which is also the radio buttons' name.
+
+=item selected - Value for the initially selected radio button.
+
+=item rows - Array ref of array refs, one per button. The elements of each are the value for this option, a label, and option additional HTML to appear next to it.
+
+=item no-bold - When set to 1, labels in the table will not be bolded
 
 =cut
 sub ui_radio_table
 {
 return &theme_ui_radio_table(@_) if (defined(&theme_ui_radio_table));
-my ($name, $sel, $rows) = @_;
+my ($name, $sel, $rows, $nobold) = @_;
 return "" if (!@$rows);
 my $rv = "<table class='ui_radio_table'>\n";
 foreach my $r (@$rows) {
        $rv .= "<tr>\n";
-       $rv .= "<td valign=top".(defined($r->[2]) ? "" : " colspan=2").
-              "><b>".
-              &ui_oneradio($name, $r->[0], $r->[1], $r->[0] eq $sel).
-              "</b></td>\n";
+       $rv .= "<td valign=top".(defined($r->[2]) ? "" : " colspan=2").">".
+              ($nobold ? "" : "<b>").
+              &ui_oneradio($name, $r->[0], $r->[1], $r->[0] eq $sel, $r->[3]).
+              ($nobold ? "" : "</b>").
+              "</td>\n";
        if (defined($r->[2])) {
                $rv .= "<td valign=top>".$r->[2]."</td>\n";
                }
@@ -1961,7 +2169,15 @@ return $rv;
 
 =head2 ui_up_down_arrows(uplink, downlink, up-show, down-show)
 
-Returns HTML for moving some objects in a table up or down
+Returns HTML for moving some objects in a table up or down. The parameters are :
+
+=item uplink - URL for the up-arrow link.
+
+=item downlink - URL for the down-arrow link.
+
+=item up-show - Set to 1 if the up-arrow should be shown, 0 if not.
+
+=item down-show - Set to 1 if the down-arrow should be shown, 0 if not.
 
 =cut
 sub ui_up_down_arrows
@@ -1989,7 +2205,7 @@ return $mover;
 
 =head2 ui_hr
 
-Returns a horizontal row tag
+Returns a horizontal row tag, typically just an <hr>
 
 =cut
 sub ui_hr
@@ -2000,7 +2216,7 @@ return "<hr>\n";
 
 =head2 ui_nav_link(direction, url, disabled)
 
-Returns an arrow icon linking to provided url
+Returns an arrow icon linking to the provided url.
 
 =cut
 sub ui_nav_link
@@ -2018,10 +2234,22 @@ else {
        }
 }
 
-=head2 ui_confirmation_form(cgi, message, &hiddens, [&buttons], [&otherinputs], [extra-warning])
+=head2 ui_confirmation_form(cgi, message, &hiddens, [&buttons], [otherinputs], [extra-warning])
 
 Returns HTML for a form asking for confirmation before performing some
-action, such as deleting a user.
+action, such as deleting a user. The parameters are :
+
+=item cgi - Script to which the confirmation form submits, like delete.cgi.
+
+=item message - Warning message for the user to see.
+
+=item hiddens - Array ref of two-element array refs, containing hidden form field names and values.
+
+=item buttons - Array ref of two-element array refs, containing form button names and labels.
+
+=item otherinputs - HTML for extra inputs to include in ther form.
+
+=item extra-warning - An additional separate warning message to show.
 
 =cut
 sub ui_confirmation_form
@@ -2047,10 +2275,10 @@ return $rv;
 
 ####################### javascript functions
 
-=head2 js_disable_inputs
+=head2 js_disable_inputs(&disable-inputs, &enable-inputs, [tag])
 
-js_disable_input(&disable-inputs, &enable-inputs, [tag])
-Returns Javascript to disable some form elements and enable others
+Returns Javascript to disable some form elements and enable others. Mainly
+for internal use.
 
 =cut
 sub js_disable_inputs
@@ -2080,7 +2308,23 @@ return $_[2] ? "$_[2]='$rv'" : $rv;
 
 Returns HTML for moving left and right in some large list, such as an inbox
 or database table. If only 5 parameters are given, no far links are included.
-If any link is undef, that array will be greyed out.
+If any link is undef, that array will be greyed out. The parameters are :
+
+=item message - Text or display between arrows.
+
+=item inputs - Additional HTML inputs to show after message.
+
+=item cgi - Optional CGI for form wrapping arrows to submit to.
+
+=item left-link - Link for left-facing arrow.
+
+=item right-link - Link for right-facing arrow.
+
+=item far-left-link - Link for far left-facing arrow, optional.
+
+=item far-right-link - Link for far right-facing arrow, optional.
+
+=item below - HTML to display below the arrows.
 
 =cut
 sub ui_page_flipper
@@ -2093,22 +2337,24 @@ $rv .= &ui_form_start($cgi) if ($cgi);
 # Far left link, if needed
 if (@_ > 5) {
        if ($farleft) {
-               $rv .= "<a href='$farleft'><img src=/images/first.gif ".
+               $rv .= "<a href='$farleft'>".
+                      "<img src=$gconfig{'webprefix'}/images/first.gif ".
                       "border=0 align=middle></a>\n";
                }
        else {
-               $rv .= "<img src=/images/first-grey.gif ".
+               $rv .= "<img src=$gconfig{'webprefix'}/images/first-grey.gif ".
                       "border=0 align=middle></a>\n";
                }
        }
 
 # Left link
 if ($left) {
-       $rv .= "<a href='$left'><img src=/images/left.gif ".
+       $rv .= "<a href='$left'>".
+              "<img src=$gconfig{'webprefix'}/images/left.gif ".
               "border=0 align=middle></a>\n";
        }
 else {
-       $rv .= "<img src=/images/left-grey.gif ".
+       $rv .= "<img src=$gconfig{'webprefix'}/images/left-grey.gif ".
               "border=0 align=middle></a>\n";
        }
 
@@ -2118,22 +2364,24 @@ $rv .= " ".$inputs if ($inputs);
 
 # Right link
 if ($right) {
-       $rv .= "<a href='$right'><img src=/images/right.gif ".
+       $rv .= "<a href='$right'>".
+              "<img src=$gconfig{'webprefix'}/images/right.gif ".
               "border=0 align=middle></a>\n";
        }
 else {
-       $rv .= "<img src=/images/right-grey.gif ".
+       $rv .= "<img src=$gconfig{'webprefix'}/images/right-grey.gif ".
               "border=0 align=middle></a>\n";
        }
 
 # Far right link, if needed
 if (@_ > 5) {
        if ($farright) {
-               $rv .= "<a href='$farright'><img src=/images/last.gif ".
+               $rv .= "<a href='$farright'>".
+                      "<img src=$gconfig{'webprefix'}/images/last.gif ".
                       "border=0 align=middle></a>\n";
                }
        else {
-               $rv .= "<img src=/images/last-grey.gif ".
+               $rv .= "<img src=$gconfig{'webprefix'}/images/last-grey.gif ".
                       "border=0 align=middle></a>\n";
                }
        }
@@ -2146,7 +2394,7 @@ return $rv;
 
 =head2 js_checkbox_disable(name, &checked-disable, &checked-enable, [tag])
 
-MISSING DOCUMENTATION
+For internal use only.
 
 =cut
 sub js_checkbox_disable
@@ -2164,7 +2412,7 @@ return $_[3] ? "$_[3]='$rv'" : $rv;
 
 =head2 js_redirect(url, [window-object])
 
-Returns HTML to trigger a redirect to some URL
+Returns HTML to trigger a redirect to some URL.
 
 =cut
 sub js_redirect
@@ -2173,6 +2421,11 @@ my ($url, $window) = @_;
 if (defined(&theme_js_redirect)) {
        return &theme_js_redirect(@_);
        }
+$window ||= "window";
+if ($url =~ /^\//) {
+       # If the URL is like /foo , add webprefix
+       $url = $gconfig{'webprefix'}.$url;
+       }
 return "<script>${window}.location = '".&quote_escape($url)."';</script>\n";
 }