From 8fc45264b932e0423d56dfab54736ce865f03871 Mon Sep 17 00:00:00 2001
From: nfreear <nfreear>
Date: Thu, 1 Nov 2007 12:32:58 +0000
Subject: [PATCH] Fixes bug MDL-11994, Accessibility: "Users tagged with ..."
 should be marked up as a list.

---
 tag/lib.php                      | 14 +++++++++-----
 theme/standard/styles_layout.css |  4 ++--
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/tag/lib.php b/tag/lib.php
index 630fabc23a..4e8113ea72 100644
--- a/tag/lib.php
+++ b/tag/lib.php
@@ -1280,9 +1280,11 @@ function print_user_list($userlist, $return=false) {
 
     $output = '';
 
+    $output .= '<ul class="inline-list">';
     foreach ($userlist as $user){
-        $output .= print_user_box( $user , true);
+        $output .= '<li>'. print_user_box( $user , true) ."</li>\n";
     }
+    $output .= "</ul>\n";
 
     if ($return) {
         return $output;
@@ -1315,15 +1317,18 @@ function print_user_box($user, $return=false) {
 
     $output .= print_box_start('user-box', 'user'.$user->id, true);
 
+    $fullname = fullname($user);
+    $alt = '';
     if (!empty($profilelink)) {
         $output .= '<a href="'.$profilelink.'">';
+        $alt = $fullname;
     }
 
-    //print user image
+    //print user image - if image is only content of link it needs ALT text!
     if ($user->picture) {
-        $output .= '<img alt="" class="user-image" src="'. $CFG->wwwroot .'/user/pix.php/'. $user->id .'/f1.jpg"'.'/>';
+        $output .= '<img alt="'.$alt.'" class="user-image" src="'. $CFG->wwwroot .'/user/pix.php/'. $user->id .'/f1.jpg"'.'/>';
     } else {
-        $output .= '<img alt="" class="user-image" src="'. $CFG->wwwroot .'/pix/u/f1.png"'.'/>';
+        $output .= '<img alt="'.$alt.'" class="user-image" src="'. $CFG->wwwroot .'/pix/u/f1.png"'.'/>';
     }
 
     $output .= '<br />';
@@ -1332,7 +1337,6 @@ function print_user_box($user, $return=false) {
         $output .= '</a>';
     }
 
-    $fullname = fullname($user);
     //truncate name if it's too big
     if ($textlib->strlen($fullname) > 26) $fullname = $textlib->substr($fullname,0,26) . '...';
 
diff --git a/theme/standard/styles_layout.css b/theme/standard/styles_layout.css
index 0d63f2fa61..2fa8caa6a9 100644
--- a/theme/standard/styles_layout.css
+++ b/theme/standard/styles_layout.css
@@ -1307,8 +1307,8 @@ a.skip:focus, a.skip:active {
   clear: left;
 }
 
-.sideblock .content .inline-list li {
-  display:inline;
+.sideblock .content .inline-list li, .inline-list li {
+  display: inline;
 }
 
 .sideblock .content ul.list li.listentry {
-- 
2.39.5