--- loncom/interface/lonwhatsnew.pm	2020/09/01 22:25:27	1.128
+++ 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.128 2020/09/01 22:25:27 raeburn Exp $
+# $Id: lonwhatsnew.pm,v 1.132 2025/03/22 21:18:30 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -118,7 +118,23 @@ sub handler {
     &Apache::loncommon::content_type($r,'text/html');
     $r->send_http_header;
 
-    $r->print(&display_header($command,\%checkallowed));
+    my $udom = $env{'user.domain'};
+    my $uname = $env{'user.name'};
+    my $cid = $env{'request.course.id'};
+    my $store_result = &store_display_settings($uname,$udom,$cid,\%checkallowed);
+    my $store_error;
+
+    unless ($store_result eq 'ok') {
+        my $lctype = lc(&Apache::loncommon::course_type());
+        &Apache::lonnet::logthis('Error saving whatsnew settings: '.
+            $store_result.' for '.'user '.$uname.':'.$udom.' in '.$lctype.' '.$cid);
+        $store_error = '<span class="LC_error">'
+                  .&mt('Unable to save visibility settings due to [_1]',
+                       $store_result)
+                  .'</span>';
+    }
+
+    $r->print(&display_header($command,$uname,$udom,$cid,\%checkallowed));
 
     &Apache::lonhtmlcommon::clear_breadcrumbs();
     &Apache::lonhtmlcommon::add_breadcrumb
@@ -185,7 +201,7 @@ sub handler {
             ("What's New?",#'Course_Action_Items_Display'
 	     ));
     }
-    &display_main_box($r,$command,$refpage,\%checkallowed,$cdom,$crs);
+    &display_main_box($r,$command,$refpage,\%checkallowed,$cdom,$crs,$store_error);
     return OK;
 }
 
@@ -196,11 +212,10 @@ sub handler {
 #------------------------------
                                                                                 
 sub display_main_box {
-    my ($r,$command,$refpage,$checkallowed,$cdom,$crs) = @_;
+    my ($r,$command,$refpage,$checkallowed,$cdom,$crs,$store_error) = @_;
     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',
@@ -245,12 +260,13 @@ sub display_main_box {
         crslogin => \%crslogins,
         sessions => \%sessions,
     );
-    my %initpage = &Apache::lonlocal::texthash (
+    my %initpage = (
                      firstres => "first resource in the $lctype",
                      whatsnew => "What's New Page",
                      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'}) {
@@ -277,15 +293,11 @@ sub display_main_box {
         &display_interval_config($r,$refpage,\%interval_titles,'sessions');
     } else {
         &display_actions_box($r,$command,$refpage,\%threshold_titles,
-                        \%interval_titles,\%initpage,$cdom,$crs,$checkallowed);
+                        \%interval_titles,\%initpage,$cdom,$crs,
+                        $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);
 }
 
 #-------------------------------
@@ -296,13 +308,48 @@ END_OF_BLOCK
 #-------------------------------
 
 sub display_header {
-    my ($command,$checkallowed) = @_;
-    
+    my ($command,$uname,$udom,$cid,$checkallowed) = @_;
+
     my $scripttag;
-    unless ($command eq 'chgthreshold' || $command eq 'chginterval' || 
-            $command eq 'chgoldroleinterval' ||
-            $command eq 'chgnewroleinterval' || $command eq 'chgcrslogininterval') {
-       $scripttag = <<"END";
+    my $args = {};
+    my %loadentries;
+
+    if ($command eq 'courseinit') {
+        $scripttag = <<"END";
+<script type="text/javascript">
+// <![CDATA[
+function toggleCourseInit(form) {
+    if (document.getElementById('courseinit_by_course')) {
+        var radioelem = form.elements['courseinit_control'];
+        if (radioelem.length) {
+            for (var i=0; i<radioelem.length; i++) {
+                if (radioelem[i].checked) {
+                    if (radioelem[i].value == 'coursespecific') {
+                        document.getElementById('courseinit_by_course').style.display = 'inline';
+                    } else {
+                         document.getElementById('courseinit_by_course').style.display = 'none';
+                    }
+                    break;
+                }
+            }
+        }
+    }
+    return;
+}
+// ]]>
+</script>
+END
+        $loadentries{'onload'} = 'toggleCourseInit(document.courseinitswitch);';
+    } else {
+        unless (($command eq 'chgthreshold' && $checkallowed->{'abovethreshold'}) ||
+                ($command eq 'chginterval' && $checkallowed->{'versionchanges'}) ||
+                ($command eq 'chgdisc' && $checkallowed->{'coursediscussion'}) ||
+                ($command eq 'chgoldroleinterval' && $checkallowed->{'oldroles'}) ||
+                ($command eq 'chgnewroleinterval' && $checkallowed->{'newroles'}) ||
+                ($command eq 'chgcrslogininterval' && $checkallowed->{'crslogin'}) ||
+                ($command eq 'chgsessionlimit' && $checkallowed->{'sessions'})) {
+            my %display_settings = &get_display_settings($uname,$udom,$cid);
+            $scripttag = <<"END";
 <script type="text/javascript">
 // <![CDATA[
 function change_display(caller,change) {
@@ -312,14 +359,14 @@ function change_display(caller,change) {
 
 function changeAll(change) {
 END
-        foreach my $item (keys(%{$checkallowed})) {
-	    if ($item =~ /_section$/) { next; }
-            if ($$checkallowed{$item}) {
-                $scripttag.='document.visible.display_'.$item.'.value=change'.
-                            "\n";
+            foreach my $item (keys(%{$checkallowed})) {
+	        if ($item =~ /_section$/) { next; }
+                if ($$checkallowed{$item}) {
+                    $scripttag.='document.visible.display_'.$item.'.value=change'.
+                                "\n";
+                }
             }
-        }
-        $scripttag.=<<"ENDTOGG";
+            $scripttag.=<<"ENDTOGG";
     document.visible.submit();
 }
 
@@ -375,10 +422,20 @@ function toggledetails(prefix) {
 // ]]>
 </script>
 ENDTOGG
+            if (($checkallowed->{'crslogin'}) && ($display_settings{$cid.':crslogin'} ne 'hide')) {
+                $loadentries{'onload'} = "toggledetails('login');";
+            }
+            if (($checkallowed->{'sessions'}) && ($display_settings{$cid.':sessions'} ne 'hide')) {
+                $loadentries{'onload'} .= "toggledetails('session');";
+            }
+        }
     }
     my $course_type=&Apache::loncommon::course_type();
+    if (keys(%loadentries)) {
+        $args->{'add_entries'} = \%loadentries;
+    }
     return &Apache::loncommon::start_page("What's New?",
-					  $scripttag);
+					  $scripttag,$args);
 }
 
 #-------------------------------
@@ -390,7 +447,7 @@ ENDTOGG
                                                                                 
 sub display_actions_box {
     my ($r,$command,$refpage,$threshold_titles,$interval_titles,$initpage,
-        $cdom,$crs,$checkallowed) = @_;
+        $cdom,$crs,$checkallowed,$store_error) = @_;
     my $udom = $env{'user.domain'};
     my $uname = $env{'user.name'};
     my $cid = $env{'request.course.id'};
@@ -467,7 +524,7 @@ sub display_actions_box {
     }
     $header .= &mt('Page set to be displayed after you have selected a role in this '.$lctype).'.'
               .' <span class="LC_nobreak">'
-              .&mt('Currently: [_1].','<i>'.$currinit.'</i>')
+              .&mt('Currently: [_1].','<span class="LC_cusr_emph">'.$currinit.'</span>')
               .'&nbsp;&nbsp;'
               .&mt('[_1]Change[_2] for just [_3]this '.$lctype.'[_4] or for [_5]all your courses/communities[_6].'
                   ,'<b>'
@@ -490,15 +547,8 @@ sub display_actions_box {
         $result = &store_discussion_setting($uname,$udom,$cid);
     }
 
-    my $store_result=&store_display_settings($uname,$udom,$cid,$checkallowed);
-
-    unless ($store_result eq 'ok') { 
-        &Apache::lonnet::logthis('Error saving whatsnew settings: '.
-            $store_result.' for '.'user '.$uname.':'.$udom.' in '.$lctype.' '.$cid);
-        $result .= '<span class="LC_error">'
-                  .&mt('Unable to save visibility settings due to [_1]',
-                       $store_result)
-                  .'</span>';
+    if ($store_error) {
+        $result = $store_error;
     }
 
     if ($result) {
@@ -690,9 +740,13 @@ sub display_actions_box {
     if ($show{'sessions'}) {
         $sessioncount = &getsessions($cdom,$crs,\%sessions,$timediff{'sessions'},$classlist);
     }
-    $r->print(qq|<a href="javascript:changeAll('hide');">$lt{'hial'}</a>
-     &nbsp;&nbsp;<a href="javascript:changeAll('show');">$lt{'shal'}</a>
-     <form method="post" name="visible" action="/adm/whatsnew">\n|);
+    my $showhideall = <<"END";
+<span class="LC_nobreak"><a href="javascript:changeAll('hide');">$lt{'hial'}</a>
+&nbsp;&nbsp;<a href="javascript:changeAll('show');">$lt{'shal'}</a></span>
+END
+    $r->print(&Apache::loncommon::head_subbox($showhideall));
+    $r->print(qq|<form method="post" name="visible" action="/adm/whatsnew">\n|);
+
     foreach my $item (keys(%{$checkallowed})) {
 	if ($item =~ /_section$/) { next; }
         if ($$checkallowed{$item}) {
@@ -700,7 +754,9 @@ sub display_actions_box {
         }
     }
 
-    $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;
@@ -716,17 +772,14 @@ sub display_actions_box {
     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>
 ');
 }
@@ -769,13 +822,18 @@ sub display_threshold_config {
         my $onchange = 'onfocus="javascript:window.document.forms'.
               "['thresholdform'].elements['".$parameter."_setparmval']".
               '.checked=true;"';
+        my $aria_textbox = ' aria-label="'.&mt('Threshold value for [_1]',
+                                               $threshold_titles{$type}).'"';
+        my $aria_checkbox = ' aria-label="'.&mt('Change threshold value for [_1]',
+                                                $threshold_titles{$type}).'"';
         $r->print(&Apache::loncommon::start_data_table_row()."\n".
                  '<td>'.$threshold_titles{$type}.'</td>'."\n".
                  '<td>'.&Apache::lonhtmlcommon::textbox($parameter.'_value',
                                             $threshold{$type},
-                                            10,$onchange).'</td>'."\n".
+                                            10,$onchange.$aria_textbox).'</td>'."\n".
                  '<td>'.
-                 &Apache::lonhtmlcommon::checkbox($parameter.'_setparmval').
+                 &Apache::lonhtmlcommon::checkbox($parameter.'_setparmval',
+                                                  '','',$aria_checkbox).
                  '</td>'."\n".
                  &Apache::loncommon::end_data_table_row());
     }
@@ -829,7 +887,7 @@ sub display_interval_config {
 <input type="hidden" name="intervaltype" value="'.$context.'" />
 <input type="hidden" name="refpage" value="'.$refpage.'" />'.
 &mt('Display:').' 
-<select name="interval">
+<select name="interval" aria-label="'.&mt('Choose time window').'">
 <option value="" selected="selected">'.&mt('Select').'</option>
 ');
     if (ref($interval_titles) eq 'HASH') {
@@ -917,46 +975,49 @@ sub courseinit_config {
     my ($control,$current) = &curr_courseinit();
     my @chgstate = ('userpref','coursespecific');
     my @chgentry = ('firstres','whatsnew');
+    my $courseinitsty = 'display:none;';
+    if ($control eq 'coursespecific') {
+        $courseinitsty = 'display:inline;';
+    }
     my $lctype = lc(&Apache::loncommon::course_type());
     my %lt = &Apache::lonlocal::texthash(
                              'chwp' => "Choose which page will be displayed when you enter this $lctype after selecting a role.",
-                             'cuva' => 'Current value is determined by',
-                             'anis' => 'and is set to display',
                              'padc' => 'Page display controlled by',
-                             'chce' => 'Choose '.$lctype.' entry',
+                             'chce' => "Choose $lctype entry",
                              'moce' => 'Save',
     );
-    $r->print(<<"END"); 
+    $lt{'cuva'} = &mt('Current value is determined by[_1]'.$initpage->{$control}.'[_2] and is set to display[_1]'.$initpage->{$current}.'[_2].',
+                      ': <span class="LC_cusr_emph">','</span>');
+    $r->print(<<"END");
 <br />$lt{'chwp'}
-<br />$lt{'cuva'}: <b>
-$$initpage{$control}</b> $lt{'anis'} <b>
-$$initpage{$current}</b>.<br /><br />
+<br />$lt{'cuva'}
+<br /><br />
 <form method="post" name="courseinitswitch" action="/adm/whatsnew">
 <input type="hidden" name="command" value="newcourseinit" />
 <input type="hidden" name="refpage" value="$refpage" />
-$lt{'padc'}:&nbsp;&nbsp;
+<fieldset style="display:inline"><legend>$lt{'padc'}</legend>
 END
     foreach my $choice (@chgstate) {
         my $chkstring;
         if ($choice eq $control) {
             $chkstring = ' checked="checked" ';
-        }  
+        }
         $r->print('<span class="LC_nobreak"><label><input type="radio" name="courseinit_control" value="'.
-                   $choice.'"'.$chkstring.'/>'.$$initpage{$choice}.
+                   $choice.'"'.$chkstring.' onclick="toggleCourseInit(this.form);" />'.&mt($initpage->{$choice}).
                    '&nbsp;&nbsp;</label></span>');
     }
-    $r->print('<br /><br />'.&mt('If').' '.$$initpage{'coursespecific'}.
-              ' - <br />'.$lt{'chce'}.": \n");
+    $r->print('</fieldset><fieldset id="courseinit_by_course" style="'.$courseinitsty.'">'.
+              '<legend>'.$lt{'chce'}.'</legend>'."\n");
     foreach my $choice (@chgentry) {
         my $chkstring;
         if (($choice eq $current) && ($control eq 'coursespecific')) {
             $chkstring = ' checked="checked" ';
         }
         $r->print('<span class="LC_nobreak"><label><input type="radio" name="courseinit_page" value="'.
-                  $choice.'"'.$chkstring.'/>'.$$initpage{$choice}.
-                  '&nbsp;&nbsp;</label></span>');
+                  $choice.'"'.$chkstring.'/>'.&mt($initpage->{$choice}).
+                  '&nbsp;&nbsp;</label></span> ');
     }
-    $r->print('<br /><br /><input type="submit" name="display" value="'.
+    $r->print('</fieldset><br /><br /><input type="submit" name="display" value="'.
                $lt{'moce'}.'" /></form>');
     return;
 }
@@ -1139,6 +1200,11 @@ sub check_handgraded {
                 if (grep(/^essay$/,@types)) {
                     $handgradeable=1;
                     last;
+                } elsif (grep(/^custom$/,@types)) {
+                    if ($resource->handgrade($part) eq 'yes') {
+                        $handgradeable=1;
+                        last;
+                    }
                 }
             }
         }
@@ -1245,8 +1311,10 @@ sub check_thresholds {
                      <td>'.$stats{$part}{degdiff}.'</td>
                      <td>'.$lastreset{$part}.'</td>';
                 if ($checkallowed->{'resetcounters'}) {
+                    my $aria = ' aria-label="'.&mt('Reset counter for [_1] (part [_2])',
+                                                  $$triggered{$symb}{title},$part).'"';
                     $$triggered{$symb}{text}[$partcount] .=
-                        '<td><input type="checkbox" name="'.$resetname.'" />'.
+                        '<td><input type="checkbox" name="'.$resetname.'"'.$aria.' />'.
                         '<input type="hidden" name="'.$resettitle.'" value="'.&escape($$triggered{$symb}{title}).'" /></td>';
                 }
                 $partcount ++;
@@ -1807,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"':'';
@@ -1832,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>');
     }
 }
 
@@ -1850,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;
 }
@@ -1885,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);
@@ -1910,12 +2005,20 @@ 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>');
-                      if (ref($$triggered{$res}{text}) eq 'ARRAY') {
-		          $r->print($$triggered{$res}{text}[0]);
-                      }
-                      $r->print('</tr>');
+		      '<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('</tr>');
             if (ref($$triggered{$res}{text}) eq 'ARRAY') {
                 if (@{$$triggered{$res}{text}} > 1) {
                     for (my $i=1; $i<@{$$triggered{$res}{text}}; $i++) {
@@ -1929,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>');
     }
 }
 
@@ -1944,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"':'';
@@ -1957,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;
 }
@@ -1984,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}}) {
@@ -2020,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>');
@@ -2030,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;
 }
@@ -2054,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}}) {
@@ -2093,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>';
                             }
@@ -2130,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;
 }
@@ -2147,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'};
@@ -2180,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>');
     }
 }
 
@@ -2201,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>');
     }
 }
 
@@ -2226,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>');
     }
 }
 
@@ -2377,7 +2557,7 @@ sub store_courseinit_setting {
                     $result = &Apache::lonhtmlcommon::confirm_success(&mt("Page displayed after role selection in $lctype now set by [_1]user's global preferences[_2].",'<b>','</b>'));
                 } else {
                     $result = &Apache::lonhtmlcommon::confirm_success(&mt('Page displayed after role selection in this '.$lctype.' set to [_1].'
-                                 ,'<b>'.$$initpage{$env{'form.courseinit_page'}}.'</b>'));
+                                 ,'<b>'.&mt($initpage->{$env{'form.courseinit_page'}}).'</b>'));
                 }
             } else {
                 &Apache::lonnet::logthis('Error saving whatsnew courseinit '.
@@ -2390,7 +2570,7 @@ sub store_courseinit_setting {
                          '<tt>'.$outcome.'</tt>'),1);
                 } else {
                     $result = &Apache::lonhtmlcommon::confirm_success(&mt('Unable to set page display, after role selection, for this '.$lctype.' to [_1] due to [_2].'
-                                 ,'<b>'.$$initpage{$env{'form.courseinit_page'}}.'</b>'
+                                 ,'<b>'.&mt($initpage->{$env{'form.courseinit_page'}}).'</b>'
                                  ,'<tt>'.$outcome.'</tt>'),1);
                 }
             }
@@ -2463,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>');
         }
     }