Index: calendar/event_new.html
===================================================================
RCS file: /home/cvs_repositories/globalcvs/ou-moodle/calendar/event_new.html,v
retrieving revision 1.5
diff -u -r1.5 event_new.html
--- calendar/event_new.html	22 Jun 2007 13:21:05 -0000	1.5
+++ calendar/event_new.html	20 Jan 2009 16:11:14 -0000
@@ -5,18 +5,18 @@
 <form method="post" action="event.php" id="eventform">
 <table cellpadding="5">
   <tr>
-    <td style="vertical-align: top; text-align: right;">
+    <td style="vertical-align: top; text-align: right;"><label for="eventname">
       <?php print_string('eventname', 'calendar'); ?>:
-    </td>
+    </label></td>
     <td>
-      <input type="text" name="name" size="67" value="<?php p($form->name); ?>" />
+      <input type="text" name="name" size="67" id="eventname" value="<?php p($form->name); ?>" />
       <?php if (isset($err['name'])) formerr($err['name']); ?>
     </td>
   </tr>
   <tr>
-    <td style="vertical-align: top; text-align: right;">
+    <td style="vertical-align: top; text-align: right;"><label for="edit-description">
       <?php print_string('eventdescription', 'calendar'); ?>:
-    </td>
+    </label></td>
     <td>
     <?php
         print_textarea($usehtmleditor, 20, 65, 630, 300, "description", $form->description);
@@ -55,8 +55,9 @@
       <div>
         <input type="radio" name="duration" value="2" id="duration_minutes" <?php if($form->duration == 2) echo 'checked="checked"'; ?>/>
         <label for="duration_minutes"><?php print_string('durationminutes', 'calendar'); ?></label>
-        <input type="text" name="minutes" size="3" value="<?php p($form->minutes); ?>" />
+        <input type="text" name="minutes" size="3" value="<?php p($form->minutes); ?>" id="minutes"/>
         <?php if (isset($err['minutes'])) formerr($err['minutes']); ?>
+        <label class="accesshide" for="minutes"><?php print_string('minutes') ?></label>
       </div>
     </td>
   </tr>
@@ -77,8 +78,8 @@
         <label for="repeat_yes">
           <?php print_string('repeatweeksl', 'calendar'); ?>
         </label>
-        <input type="text" name="repeats" size="2" value="<?php p($form->repeats); ?>" />
-        <?php print_string('repeatweeksr', 'calendar'); ?>
+        <input id="repeats" type="text" name="repeats" size="2" value="<?php p($form->repeats); ?>" />
+        <label for="repeats"><?php print_string('repeatweeksr', 'calendar'); ?></label>
         <?php if (isset($err['repeats'])) formerr($err['repeats']); ?>
       </div>
     </td>
Index: lib/weblib.php
===================================================================
RCS file: /home/cvs_repositories/globalcvs/ou-moodle/lib/weblib.php,v
retrieving revision 1.185
diff -u -r1.185 weblib.php
--- lib/weblib.php	10 Dec 2008 10:54:15 -0000	1.185
+++ lib/weblib.php	20 Jan 2009 16:11:14 -0000
@@ -6225,10 +6225,24 @@
     for ($i=1970; $i<=2020; $i++) {
         $years[$i] = $i;
     }
-    return choose_from_menu($days,   $day,   $currentdate['mday'], '', '', '0', $return)
-          .choose_from_menu($months, $month, $currentdate['mon'],  '', '', '0', $return)
-          .choose_from_menu($years,  $year,  $currentdate['year'], '', '', '0', $return);
 
+    // Build or print result
+    $result='';
+    // Note: There should probably be a fieldset around these fields as they are
+    // clearly grouped. However this causes problems with display. See Mozilla
+    // bug 474415
+    $result.='<label class="accesshide" for="menu'.$day.'">'.get_string('day','form').'</label>';
+    $result.=choose_from_menu($days,   $day,   $currentdate['mday'], '', '', '0', true);
+    $result.='<label class="accesshide" for="menu'.$month.'">'.get_string('month','form').'</label>';
+    $result.=choose_from_menu($months, $month, $currentdate['mon'],  '', '', '0', true);
+    $result.='<label class="accesshide" for="menu'.$year.'">'.get_string('year','form').'</label>';
+    $result.=choose_from_menu($years,  $year,  $currentdate['year'], '', '', '0', true);
+
+    if ($return) {
+        return $result;
+    } else {
+        echo $result;
+    }
 }
 
 /**
@@ -6256,8 +6270,21 @@
         $minutes[$i] = sprintf("%02d",$i);
     }
 
-    return choose_from_menu($hours,   $hour,   $currentdate['hours'],   '','','0',$return)
-          .choose_from_menu($minutes, $minute, $currentdate['minutes'], '','','0',$return);
+    // Build or print result
+    $result='';
+    // Note: There should probably be a fieldset around these fields as they are
+    // clearly grouped. However this causes problems with display. See Mozilla
+    // bug 474415
+    $result.='<label class="accesshide" for="menu'.$hour.'">'.get_string('hour','form').'</label>';
+    $result.=choose_from_menu($hours,   $hour,   $currentdate['hours'],   '','','0',true);
+    $result.='<label class="accesshide" for="menu'.$minute.'">'.get_string('minute','form').'</label>';
+    $result.=choose_from_menu($minutes, $minute, $currentdate['minutes'], '','','0',true);
+
+    if ($return) {
+        return $result;
+    } else {
+        echo $result;
+    }
 }
 
 /**