--- loncom/interface/lonwhatsnew.pm	2025/03/22 20:34:49	1.131
+++ loncom/interface/lonwhatsnew.pm	2025/03/22 21:18:30	1.132
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # What's New in a course
 #
-# $Id: lonwhatsnew.pm,v 1.131 2025/03/22 20:34:49 raeburn Exp $
+# $Id: lonwhatsnew.pm,v 1.132 2025/03/22 21:18:30 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -216,7 +216,6 @@ sub display_main_box {
     my $domain=&Apache::loncommon::determinedomain();
     my $function = &Apache::loncommon::get_users_function();
     my $lctype = lc(&Apache::loncommon::course_type());
-    $r->print('<table width="100%" border="0" cellpadding="5" cellspacing="0"><tr><td width="100%">');
 
     my %threshold_titles = &Apache::lonlocal::texthash (
                          av_attempts => 'Average number of attempts',
@@ -267,6 +266,7 @@ sub display_main_box {
                      userpref => 'your general user preferences',
                      coursespecific => "specific setting for this $lctype",
                    );
+    $r->print('<div class="LC_landmark" role="main">');
 
     if (($command eq 'chgthreshold') 
 	&& $checkallowed->{'abovethreshold'}) {
@@ -297,12 +297,7 @@ sub display_main_box {
                         $checkallowed,$store_error);
     }
     my $end_page = &Apache::loncommon::end_page();
-    $r->print(<<END_OF_BLOCK);
-  </td>
- </tr>
-</table><br />
-$end_page
-END_OF_BLOCK
+    $r->print('<br /></div>'.$end_page);
 }
 
 #-------------------------------
@@ -759,7 +754,9 @@ END
         }
     }
 
-    $r->print('<input type="hidden" name="refpage" value="'.$refpage.'" /><table class="LC_double_column"><tr><td class="LC_left_col">');
+    $r->print('<input type="hidden" name="refpage" value="'.$refpage.'" />'."\n".
+              '<div class=" LC_landmark" style="clear: both; width: 100%">'."\n".
+              '<div class="LC_floatleft LC_landmark" style="width: 48%">');
 
     my $displayed = 0;
     my $totalboxes = 0;
@@ -775,17 +772,14 @@ END
     foreach my $actionitem (@actionorder) {
         if ($checkallowed->{$actionitem}) {
             if (($actioncolumn{$actionitem} eq 'right') && ($currcolumn eq 'left')) {
-                $r->print('</td><td>&nbsp;</td><td class="LC_right_col" >');
-                $currcolumn = 'right'; 
+                $r->print('</div><div class="LC_floatright LC_landmark" style="width: 48%">');
+                $currcolumn = 'right';
             }
             &display_launcher($r,$actionitem,$refpage,$checkallowed,\%show,\%headings,\%res_title,\@tograde,\%ungraded,\@bombs,\%bombed,\%changed,\@warnings,\%triggered,\@newdiscussions,\%unread,$msgcount,\@newmsgs,$critmsgcount,\@critmsgs,\%interval,$countunread,\%expired,$expirecount,\%activated,$activecount,$crstype,$itemserror,\%loggedin,$logincount,\%sessions,$sessioncount,$classlist);
             $displayed ++; 
         }
     }
-    $r->print('
-      </td>
-    </tr>
-   </table>
+    $r->print('</div></div><div style="padding:0;clear:both;margin:0;border:0"></div>
    </form>
 ');
 }
@@ -1881,10 +1875,14 @@ sub display_handgrade {
                         'nopr' => 'No problems require handgrading',
     );
     if (@{$tograde} > 0) {
-        $r->print('<tr class="LC_info_row"><td class="LC_left_item">'.
-                  $lt{'prna'}.'</td><td class="LC_right_item">'.
-                  $lt{'nmun'}.'</td></tr>');
+        $r->print('<tr class="LC_info_row"><th class="LC_left_item">'.
+                  $lt{'prna'}.'</th><th class="LC_right_item">'.
+                  $lt{'nmun'}.'</th></tr>');
         my $rowNum = 0;
+        my $usediv; 
+        if (@{$tograde} > 1) {
+            $usediv = 1;
+        } 
         foreach my $res (@{$tograde}) {
             $rowNum ++;
             my $css_class = $rowNum%2?' class="LC_odd_row"':'';
@@ -1906,12 +1904,21 @@ sub display_handgrade {
                 }
                 $linkurl.='&amp;command=ungraded';
             }
-            $r->print('<tr'.$css_class.'><td><a href="'.$linkurl.'">'.$$ungraded{$res}{title}.'</a></td><td class="LC_right_item">'.$$ungraded{$res}{count}.'</td></tr>');
+            $r->print('<tr'.$css_class.'><td>');
+            if ($usediv) {
+                $r->print('<div class="LC_minheight">');
+            }
+            $r->print('<a href="'.$linkurl.'">'.$$ungraded{$res}{title}.'</a>');
+            if ($usediv) {
+                $r->print('</div>');
+            }
+            $r->print('</td>'.
+                      '<td class="LC_right_item">'.$$ungraded{$res}{count}.'</td></tr>');
         }
     } elsif ($itemserror) {
-        $r->print('<tr class="LC_empty_row"><td>'.$itemserror.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$itemserror.'</th></tr>');
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'.$lt{'nopr'}.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$lt{'nopr'}.'</th></tr>');
     }
 }
 
@@ -1924,21 +1931,32 @@ sub display_haserrors {
                                    noer => 'No problems with errors',
     );
     if (@{$bombs} > 0) {
-        $r->print('<tr class="LC_info_row"><td class="LC_left_item">'.
-                  $lt{'reso'}.'</td><td class="LC_right_item">'.
-                  $lt{'nmer'}.'</td></tr>');
+        $r->print('<tr class="LC_info_row"><th class="LC_left_item">'.
+                  $lt{'reso'}.'</th><th class="LC_right_item">'.
+                  $lt{'nmer'}.'</th></tr>');
         @{$bombs} = sort { &cmp_title($a,$b,$res_title) } @{$bombs};
+        my $usediv;
+        if (@{$bombs} > 1) {
+            $usediv = 1;
+        }
         foreach my $bomb (@{$bombs}) {
             $bombnum ++;
             my $css_class = $bombnum%2?' class="LC_odd_row"':'';
-            $r->print('<tr'.$css_class.'><td>'.$$bombed{$bomb}{errorlink}.
-                      '</td><td class="LC_right_item">'.
+            $r->print('<tr'.$css_class.'><td style="vertical-align:middle;">');
+            if ($usediv) {
+                $r->print('<div class="LC_minheight">');
+            }
+            $r->print($$bombed{$bomb}{errorlink});
+            if ($usediv) {
+                $r->print('</div>');
+            } 
+            $r->print('</td><td class="LC_right_item" style="vertical-align:middle;">'.
                       $$bombed{$bomb}{errorcount}.'</td></tr>');
         }
     } elsif ($itemserror) {
-        $r->print('<tr class="LC_empty_row"><td>'.$itemserror.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$itemserror.'</th></tr>');
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'.$lt{'noer'}.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$lt{'noer'}.'</th></tr>');
     }
     return;
 }
@@ -1959,16 +1977,19 @@ sub display_abovethreshold {
     if (@{$warnings} > 0) {
         @{$warnings} = sort { &cmp_title($a,$b,$res_title) } @{$warnings};
         $r->print('<tr class="LC_info_row">'.
-		  '<td class="LC_left_item">'.$lt{'reso'}.
-                  '<input type="hidden" name="command" value="" /></td>'.
-		  '<td>'.$lt{'part'}.'</td><td>'.$lt{'nust'}.'</td>'.
-		  '<td>'.$lt{'avat'}.'</td><td>'.$lt{'dedi'}.'</td>'.
-		  '<td>'.$lt{'lare'}.'</td>');
+		  '<th class="LC_left_item">'.$lt{'reso'}.
+                  '<input type="hidden" name="command" value="" /></th>'.
+		  '<th>'.$lt{'part'}.'</th><th>'.$lt{'nust'}.'</th>'.
+		  '<th>'.$lt{'avat'}.'</th><th>'.$lt{'dedi'}.'</th>'.
+		  '<th>'.$lt{'lare'}.'</th>');
         if ($checkallowed->{'resetcounters'}) {
-            $r->print('<td  class="LC_right_item">'.$lt{'reco'}.'</td>');
+            $r->print('<th class="LC_right_item">'.$lt{'reco'}.'</th>');
         }
         $r->print('</tr>');
-	my $row;
+	my ($usediv,$row);
+        if (@{$warnings} > 1) {
+            $usediv = 1;
+        }
         foreach my $res (@{$warnings}) {
 	    $row++;
             my ($map,$id,$url)=&Apache::lonnet::decode_symb($res);
@@ -1984,10 +2005,18 @@ sub display_abovethreshold {
             }
             my $css_class = $row%2?' class="LC_odd_row"':'';
             $r->print('<tr'.$css_class.'>'.
-		      '<td class="LC_first_item" '.$rowspan.'><a href="'.$linkurl.'">'.
-		      $$triggered{$res}{title}.'</a></td>');
+		      '<td class="LC_first_item" '.$rowspan.'>');
+            if ($usediv) {
+                $r->print('<div class="LC_minheight">');
+            }
+            $r->print('<a href="'.$linkurl.'">'.
+		      $$triggered{$res}{title}.'</a>');
+            if ($usediv) {
+                $r->print('</div>');
+            }
+            $r->print('</td>');
             if (ref($$triggered{$res}{text}) eq 'ARRAY') {
-                $r->print($$triggered{$res}{text}[0]);
+	        $r->print($$triggered{$res}{text}[0]);
             }
             $r->print('</tr>');
             if (ref($$triggered{$res}{text}) eq 'ARRAY') {
@@ -2003,9 +2032,9 @@ sub display_abovethreshold {
             $r->print('<tr class="LC_info_row"><td colspan="7" class="LC_right_item"><br /><input type="button" name="counters" value="'.$lt{'rese'}.'" onclick="javascript:thresholdreset();" /></td></tr>');
         }
     } elsif ($itemserror) {
-        $r->print('<tr class="LC_empty_row"><td>'.$itemserror.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$itemserror.'</th></tr>');
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'.$lt{'nopr'}.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$lt{'nopr'}.'</th></tr>');
     }
 }
 
@@ -2018,12 +2047,18 @@ sub display_versionchanges {
         'veru' => 'Version used',
     );
     if (keys(%{$changed}) > 0) {
-        $r->print('<tr class="LC_info_row"><td class="LC_left_item">'.
-                  $lt{'reso'}.'</td><td>'.$lt{'revd'}.'</td><td>'.
-                  $lt{'newv'}.'</td><td class="LC_right_item">'.
-                  $lt{'veru'}.'</td></tr>');
+        $r->print('<tr class="LC_info_row">'.
+                  '<th class="LC_left_item LC_nobreak">'.$lt{'reso'}.'</th>'.
+                  '<th class="LC_left_item LC_nobreak">'.$lt{'revd'}.'</th>'.
+                  '<th class="LC_left_item LC_nobreak">'.$lt{'newv'}.'</th>'.
+                  '<th class="LC_left_item LC_nobreak">'.$lt{'veru'}.'</th>'.
+                  '</tr>');
         my @changes = sort { &cmp_title($a,$b,$res_title) } keys(%{$changed});
         my $changenum = 0;
+        my $usediv;
+        if (@changes > 1) {
+            $usediv = 1;
+        }
         foreach my $item (@changes) {
             $changenum ++;
             my $css_class = $changenum%2?' class="LC_odd_row"':'';
@@ -2031,17 +2066,25 @@ sub display_versionchanges {
             my $linkurl=&Apache::lonnet::clutter($url);
             $linkurl .= '?symb='.&escape($item);
 
-            $r->print('<tr'.$css_class.'><td><a href="'.$linkurl.'">'.
-                      $$res_title{$item}.'</a></td><td>'.
+            $r->print('<tr'.$css_class.'><td>');
+            if ($usediv) {
+                $r->print('<div class="LC_minheight">');
+            }
+            $r->print('<a href="'.$linkurl.'">'.
+                      $$res_title{$item}.'</a>');
+            if ($usediv) {
+                $r->print('</div>');
+            }
+            $r->print('</td><td>'.
                       $$changed{$item}{'revdate'}.'</td><td>'.
                       $$changed{$item}{'current'}.'</td><td>'.
                       $$changed{$item}{'version'}.'</td></tr>');
         }
     } elsif ($itemserror) {
-        $r->print('<tr class="LC_empty_row"><td>'.$itemserror.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$itemserror.'</th></tr>');
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'
-                 .&mt('No '.$interval).'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'
+                 .&mt('No '.$interval).'</th></tr>');
     }
     return;
 }
@@ -2058,15 +2101,19 @@ sub display_rolechanges {
     );
     if ($chgcount) {
         $r->print('<tr class="LC_info_row">'.
-                  '<td class="LC_left_item">'.$lt{'tich'}.'</td>'.
-                  '<td class="LC_left_item">'.$lt{'user'}.'</td>'.
-                  '<td class="LC_left_item">'.$lt{'role'}.'</td>'.
-                  '<td class="LC_left_item">'.$lt{'sec'}.'</td>'.
-                  '<td class="LC_left_item">'.$lt{'status'}.'</td></tr>');
+                  '<th class="LC_left_item">'.$lt{'tich'}.'</th>'.
+                  '<th class="LC_left_item">'.$lt{'user'}.'</th>'.
+                  '<th class="LC_left_item">'.$lt{'role'}.'</th>'.
+                  '<th class="LC_left_item">'.$lt{'sec'}.'</th>'.
+                  '<th class="LC_left_item">'.$lt{'status'}.'</th></tr>');
         if (ref($changed) eq 'HASH') {
             my @changes = sort { $b <=> $a } (keys(%{$changed}));
             my $changenum = 0;
             my $fullnameidx = &Apache::loncoursedata::CL_FULLNAME();
+            my $usediv;
+            if ($chgcount > 1) {
+                $usediv = 1;
+            }
             foreach my $item (@changes) {
                 if (ref($changed->{$item}) eq 'ARRAY') {
                     foreach my $chg (@{$changed->{$item}}) {
@@ -2094,7 +2141,15 @@ sub display_rolechanges {
                                 &Apache::loncommon::aboutmewrapper($fullname,$uname,$udom);
                             $r->print('<tr'.$css_class.'>'.
                                       '<td>'.&Apache::lonlocal::locallocaltime($item).'</td>'.
-                                      '<td>'.$link.'</td>'.
+                                      '<td>');
+                            if ($usediv) {
+                                $r->print('<div class="LC_minheight">');
+                            }
+                            $r->print($link);
+                            if ($usediv) {
+                                $r->print('</div>');
+                            }
+                            $r->print('</td>'.
                                       '<td>'.$role.'</td>'.
                                       '<td>'.$section.'</td>'.
                                       '<td>'.$status.'</td></tr>');
@@ -2104,8 +2159,8 @@ sub display_rolechanges {
             }
         }
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'
-                 .&mt('There are no '.$interval).'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'
+                 .&mt('There are no '.$interval).'</th></tr>');
     }
     return;
 }
@@ -2128,22 +2183,26 @@ sub display_activity {
     if ($count) {
 
         my $hdr = '<tr class="LC_info_row" style="display:none" id="'.$prefix.'titledet">'.
-                  '<td class="LC_left_item">'.$lt{'user'}.'</td>'.
-                  '<td class="LC_left_item">'.$lt{'role'}.'</td>'.
-                  '<td class="LC_left_item">'.$lt{'sec'}.'</td>';
+                  '<th class="LC_left_item">'.$lt{'user'}.'</th>'.
+                  '<th class="LC_left_item">'.$lt{'role'}.'</th>'.
+                  '<th class="LC_left_item">'.$lt{'sec'}.'</th>';
         if ($context eq 'sessions') {
-            $hdr .= '<td class="LC_left_item">'.$lt{'active'}.'</td>';
+            $hdr .= '<th class="LC_left_item">'.$lt{'active'}.'</th>';
         }
         $hdr .= '</tr>'."\n".
                 '<tr class="LC_info_row" id="'.$prefix.'titlesum">'.
-                '<td class="LC_left_item">'.$lt{'number'}.'</td>'.
-                '<td class="LC_left_item">'.$lt{'role'}.'</td>'.
-                '<td class="LC_left_item">'.$lt{'sec'};
+                '<th class="LC_left_item">'.$lt{'number'}.'</th>'.
+                '<th class="LC_left_item">'.$lt{'role'}.'</th>'.
+                '<th class="LC_left_item">'.$lt{'sec'};
         my (%bylastname,%counts);
         if (ref($details) eq 'HASH') {
             my @items = sort { $b <=> $a } (keys(%{$details}));
             my $num = 0;
             my $fullnameidx = &Apache::loncoursedata::CL_FULLNAME();
+            my $usediv;
+            if ($count > 1) {
+                $usediv = 1;
+            }
             foreach my $item (@items) {
                 if (ref($details->{$item}) eq 'ARRAY') {
                     foreach my $user (@{$details->{$item}}) {
@@ -2167,9 +2226,17 @@ sub display_activity {
                             }
                             my $link =
                                 &Apache::loncommon::aboutmewrapper($fullname,$uname,$udom);
-                            my $entry = '<td>'.$link.'</td>'.
-                                        '<td>'.$role.'</td>'.
-                                        '<td>'.$section.'</td>';
+                            my $entry = '<td>';
+                            if ($usediv) {
+                                $entry .= '<div class="LC_minheight">';
+                            }
+                            $entry .= $link;
+                            if ($usediv) {
+                                $entry .= '</div>';
+                            }
+                            $entry .= '</td>'.
+                                      '<td>'.$role.'</td>'.
+                                      '<td>'.$section.'</td>';
                             if ($context eq 'sessions') {
                                 $entry .= '<td>'.&Apache::lonlocal::locallocaltime($item).'</td>';
                             }
@@ -2204,13 +2271,13 @@ sub display_activity {
             }
             $r->print($hdr.'<input type="hidden" name="'.$prefix.'count" value="'.$num.
                       '" /><input type="hidden" name="'.$prefix.'row" value="'.$numrow.
-                      '" /></td></tr>'.$table);
+                      '" /></th></tr>'.$table);
         }
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'.
+        $r->print('<tr class="LC_empty_row"><th>'.
                   &mt('There are no '.$interval).
                   '<input type="hidden" name="'.$prefix.'count" value="'.$count.
-                  '." /></td></tr>');
+                  '." /></th></tr>');
     }
     return;
 }
@@ -2221,24 +2288,28 @@ sub display_coursediscussion {
     my %lt = &Apache::lonlocal::texthash(
                 'loca' => 'Location',
                 'type' => 'Type',
-                'numn' => 'Number of new posts',
+                'numn' => 'New posts',
                 'noun' => 'No unread posts in '.$lctype.' discussions',
                 'tmlp' => 'Time of last post', 
     );
     if (@{$newdiscussions} > 0) {
-        $r->print('<tr class="LC_info_row"><td class="LC_left_item">'.
-                  $lt{'loca'}.'</td><td>'.
-                  $lt{'type'}.'</td>');
+        $r->print('<tr class="LC_info_row">'.
+                  '<th class="LC_left_item LC_nobreak">'.$lt{'loca'}.'</th>'.
+                  '<th class="LC_left_item LC_nobreak">'.$lt{'type'}.'</th>');
         if ($countunread eq 'on') {
-            $r->print('<td>'.$lt{'tmlp'}.'</td>'.
-                      '<td class="LC_right_item">'.$lt{'numn'}.'</td>');
+            $r->print('<th class="LC_left_item LC_nobreak">'.$lt{'tmlp'}.'</th>'.
+                      '<th class="LC_left_item LC_nobreak">'.$lt{'numn'}.'</th>');
         } else {
-            $r->print('<td class="LC_right_item">'.$lt{'tmlp'}.'</td>');
+            $r->print('<th class="LC_left_item">'.$lt{'tmlp'}.'</th>');
         }
         $r->print("</tr>\n");
         @{$newdiscussions} = sort { &cmp_title($a,$b,$res_title) }
                                                             @{$newdiscussions};
         my $rowNum = 0;
+        my $usediv;
+        if (@{$newdiscussions} > 1) {
+            $usediv = 1;
+        }
         foreach my $ressymb (@{$newdiscussions}) {
             $rowNum ++;
             my $forum_title = $$unread{$ressymb}{'title'};
@@ -2254,20 +2325,29 @@ sub display_coursediscussion {
             my $css_class = $rowNum%2?' class="LC_odd_row"':'';
             my $lastpost = &Apache::lonnavmaps::timeToHumanString(
                                                $$unread{$ressymb}{'lastpost'});
-            $r->print('<tr'.$css_class.'><td><a href="'.$disclink.'">'.$forum_title.'</a>&nbsp;</td><td>'.&mt($type).'&nbsp;</td>');
+            $r->print('<tr'.$css_class.'><td>');
+            if ($usediv) {
+                $r->print('<div class="LC_minheight">');
+            }
+            $r->print('<a href="'.$disclink.'">'.$forum_title.'</a>&nbsp;');
+            if ($usediv) {
+                $r->print('</div>');
+            }
+            $r->print('</td>'.
+                      '<td>'.&mt($type).'&nbsp;</td>');
             if ($countunread eq 'on') {
                 my $unreadnum = $$unread{$ressymb}{'unreadcount'};
-                $r->print('<td>'.$lastpost.'</td><td class="LC_right_item">'.
-                          $unreadnum.'&nbsp;</td>');
+                $r->print('<td>'.$lastpost.'</td><td class="LC_left_item">'.
+                          $unreadnum.'</td>');
             } else {
-                $r->print('<td class="LC_right_item">'.$lastpost.'</td>');
+                $r->print('<td class="LC_left_item">'.$lastpost.'</td>');
             }
             $r->print("</tr>\n");
         }
     } elsif ($itemserror) {
-        $r->print('<tr class="LC_empty_row"><td>'.$itemserror.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$itemserror.'</th></tr>');
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'.$lt{'noun'}.'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.$lt{'noun'}.'</th></tr>');
     }
 }
 
@@ -2275,24 +2355,37 @@ sub display_coursenormalmail {
     my ($r,$msgcount,$newmsgs) = @_;
     my $lctype = lc(&Apache::loncommon::course_type());
     if ($msgcount > 0) {
-        $r->print('<tr class="LC_info_row"><td class="LC_left_item">'.
-                  &mt('Number').'</td><td>'.&mt('Subject').'</td><td>'.
-                  &mt('Sender').'</td><td class="LC_right_item">'.
-                  &mt('Date/Time').'</td></tr>');
+        $r->print('<tr class="LC_info_row"><th class="LC_left_item">'.
+                  &mt('Number').'&nbsp;</th><th class="LC_left_item">'.
+                  &mt('Subject').'</th><th class="LC_left_item">'.
+                  &mt('Sender').'</th><th class="LC_left_item">'.
+                  &mt('Date/Time').'</th></tr>');
         my $mailcount = 0;
+        my $usediv;
+        if (@{$newmsgs}) {
+            $usediv = 1;
+        }
         foreach my $msg (@{$newmsgs}) {
             $mailcount ++;
             my $css_class = $mailcount%2?' class="LC_odd_row"':'';
             $r->print('<tr'.$css_class.'><td>'.$mailcount
-                      .'.&nbsp;</td><td><a href="/adm/email?display='
+                      .'.&nbsp;</td><td>');
+            if ($usediv) {
+                $r->print('<div class="LC_minheight">');
+            }
+            $r->print('<a href="/adm/email?display='
                       .$msg->{'msgid'}.'">'
-                      .$msg->{'shortsub'}.'</a>&nbsp;&nbsp;</td><td>&nbsp;'
-                      .$msg->{'from'}.':'.$msg->{'fromdom'}.'&nbsp;</td><td>'
+                      .$msg->{'shortsub'}.'</a>&nbsp;&nbsp;');
+            if ($usediv) {
+                $r->print('</div>');
+            }
+            $r->print('</td><td>'
+                      .$msg->{'from'}.':'.$msg->{'fromdom'}.'&nbsp;&nbsp;</td><td>'
                       .$msg->{'sendtime'}.'</td></tr>');
         }
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'.
-                  &mt('No new '.$lctype.' messages').'</td></tr>');
+        $r->print('<tr class="LC_empty_row"><th>'.
+                  &mt('No new '.$lctype.' messages').'</th></tr>');
     }
 }
 
@@ -2300,24 +2393,37 @@ sub display_coursecritmail {
     my ($r,$critmsgcount,$critmsgs) = @_;
     my $lctype = lc(&Apache::loncommon::course_type());
     if ($critmsgcount > 0) {
-        $r->print('<tr class="LC_info_row"><td class="LC_left_item">'.
-                  &mt('Number').'</td><td>'.&mt('Subject').'</td><td>'.
-                  &mt('Sender').'</td><td class="LC_right_item">'.
-                  &mt('Date/Time').'</td></tr>');
+        $r->print('<tr class="LC_info_row"><th class="LC_left_item">'.
+                  &mt('Number').'</th><th>'.&mt('Subject').'</th><th>'.
+                  &mt('Sender').'</th><th class="LC_right_item">'.
+                  &mt('Date/Time').'</th></tr>');
         my $mailcount = 0;
+        my $usediv;
+        if (@{$critmsgs} > 1) {
+            $usediv = 1;
+        }
         foreach my $msg (@{$critmsgs}) {
             $mailcount ++;
             my $css_class = $mailcount%2?' class="LC_odd_row"':'';
             $r->print('<tr'.$css_class.'><td>'.$mailcount.
-                      '. &nbsp;</td><td><a href="/adm/email?folder=critical">'.
-                      $msg->{'shortsub'}.'</a>&nbsp; &nbsp;</td><td>&nbsp;'.
+                      '. &nbsp;</td>'.
+                      '<td>');
+            if ($usediv) {
+                $r->print('<div class="LC_minheight">');
+            }
+            $r->print('<a href="/adm/email?folder=critical">'.
+                      $msg->{'shortsub'}.'</a>&nbsp; &nbsp;');
+            if ($usediv) {
+                $r->print('</div>');
+            }
+            $r->print('</td><td>&nbsp;'.
                       $msg->{'from'}.':'.$msg->{'fromdom'}.'&nbsp;</td><td>'.
                       $msg->{'sendtime'}.'</td></tr>');
         }
     } else {
-        $r->print('<tr class="LC_empty_row"><td>'.
+        $r->print('<tr class="LC_empty_row"><th>'.
                   &mt('No unread critical messages in '.$lctype).
-                  '</td></tr>');
+                  '</th></tr>');
     }
 }
 
@@ -2537,14 +2643,14 @@ sub start_box {
         if ($$show{$caller}) {
             $r->print('
               <tr>
-                <td class="LC_subheader" align="left"><span class="LC_nobreak"><label><input type="radio" name="logindetails" value="0" checked="checked" onclick="javascript:toggledetails('."'login'".');" />&nbsp;'.&mt('Summary').'</label><label><input type="radio" name="logindetails" value="1" onclick="javascript:toggledetails('."'login'".');" />&nbsp;'.&mt('Details').'</label></span></td><td class="LC_subheader"><a href="/adm/whatsnew?command=chgcrslogininterval&amp;refpage='.$refpage.'">'.$lt{'chin'}.'</a></td>
+                <td class="LC_subheader" align="left"><fieldset class="LC_landmark"><legend><span class="LC_visually_hidden">'.&mt('Go').'</span></legend><span class="LC_nobreak"><label><input type="radio" name="logindetails" value="0" checked="checked" onclick="javascript:toggledetails('."'login'".');" />&nbsp;'.&mt('Summary').'</label><label><input type="radio" name="logindetails" value="1" onclick="javascript:toggledetails('."'login'".');" />&nbsp;'.&mt('Details').'</label></span></fieldset></td><td class="LC_subheader"><a href="/adm/whatsnew?command=chgcrslogininterval&amp;refpage='.$refpage.'">'.$lt{'chin'}.'</a></td>
               </tr>');
         }
     } elsif (($caller eq 'sessions') && ($$show{$caller})) {
         if ($$show{$caller}) {
             $r->print('
               <tr>
-                <td class="LC_subheader" align="left"><span class="LC_nobreak"><label><input type="radio" name="sessiondetails" value="0" checked="checked" onclick="javascript:toggledetails('."'session'".');" />&nbsp;'.&mt('Summary').'</label><label><input type="radio" name="sessiondetails" value="1" onclick="javascript:toggledetails('."'session'".');" />&nbsp;'.&mt('Details').'</label></span></td><td class="LC_subheader"><a href="/adm/whatsnew?command=chgsessionlimit&amp;refpage='.$refpage.'">'.$lt{'chin'}.'</a></td>
+                <td class="LC_subheader" align="left"><fieldset class="LC_landmark"><legend><span class="LC_visually_hidden">'.&mt('Leave').'</span></legend><span class="LC_nobreak"><label><input type="radio" name="sessiondetails" value="0" checked="checked" onclick="javascript:toggledetails('."'session'".');" />&nbsp;'.&mt('Summary').'</label><label><input type="radio" name="sessiondetails" value="1" onclick="javascript:toggledetails('."'session'".');" />&nbsp;'.&mt('Details').'</label></span></td><td class="LC_subheader"><a href="/adm/whatsnew?command=chgsessionlimit&amp;refpage='.$refpage.'">'.$lt{'chin'}.'</a></td>
               </tr>');
         }
     }