From d53f1cdb82099ed6d4247a041b73b5d00684858d Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Sat, 30 Dec 2006 11:20:26 +0000
Subject: [PATCH] MDL-7948 some more messaging accessiblity fixes

---
 message/discussion.php | 4 ++++
 message/lib.php        | 5 +++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/message/discussion.php b/message/discussion.php
index e32394c731..3c7bc06257 100644
--- a/message/discussion.php
+++ b/message/discussion.php
@@ -281,9 +281,13 @@
     if (empty($allmessages)) {
         echo get_string('nomessagesfound', 'message');
     } else {
+        echo '<ul class="messagelist">';
         foreach ($allmessages as $message) {
+            echo '<li>';
             echo $message;
+            echo '</li>';
         }
+        echo '</ul>';
         if ($playbeep and get_user_preferences('message_beepnewmessage', 0)) {
             echo '<embed src="bell.wav" autostart="true" hidden="true" name="bell" />';
         }
diff --git a/message/lib.php b/message/lib.php
index 04b3672bc1..f5498f4bf2 100644
--- a/message/lib.php
+++ b/message/lib.php
@@ -586,6 +586,7 @@ function message_contact_link($userid, $linktype='add', $return=false, $script="
 
     $command = $linktype.'contact';
     $string  = $str->{$command};
+    $alttext = $text ? '' : $string; 
     $text = $text ? '&nbsp;'.$string : '';
 
     switch ($linktype) {
@@ -606,7 +607,7 @@ function message_contact_link($userid, $linktype='add', $return=false, $script="
     $output = '<span class="'.$linktype.'">'.
               '<a href="'.$script.'&amp;'.$command.'='.$userid.
               '&amp;sesskey='.sesskey().'" title="'.s($string).'">'.
-              '<img src="'.$CFG->pixpath.$icon.'" height="11" width="11" border="0" alt="'.s($string).'" />'.
+              '<img src="'.$CFG->pixpath.$icon.'" height="11" width="11" border="0" alt="'.s($alttext).'" />'.
               $text.'</a></span>';
 
     if ($return) {
@@ -639,7 +640,7 @@ function message_history_link($userid1, $userid2=0, $returnstr=false, $keywords=
     if ($linktext == 'icon') {  // Icon only
         $fulllink = '<img src="'.$CFG->pixpath.'/t/log.gif" height="11" width="11" border="0" alt="'.$strmessagehistory.'" />';
     } else if ($linktext == 'both') {  // Icon and standard name
-        $fulllink = '<img src="'.$CFG->pixpath.'/t/log.gif" height="11" width="11" border="0" alt="'.$strmessagehistory.'" />';
+        $fulllink = '<img src="'.$CFG->pixpath.'/t/log.gif" height="11" width="11" border="0" alt="" />';
         $fulllink .= '&nbsp;'.$strmessagehistory;
     } else if ($linktext) {    // Custom name
         $fulllink = $linktext;
-- 
2.39.5