MDL-12192 temporarily disabled support for colonized labels
authorDavid Mudrak <david.mudrak@gmail.com>
Fri, 24 Sep 2010 11:19:57 +0000 (11:19 +0000)
committerDavid Mudrak <david.mudrak@gmail.com>
Fri, 24 Sep 2010 11:19:57 +0000 (11:19 +0000)
The code must be fixed so that the colon is not displayed when the label
is hidden by accessibility CSS.

lang/en/langconfig.php
lib/outputcomponents.php

index c23bd5c..7905f66 100644 (file)
@@ -29,7 +29,7 @@ $string['decsep'] = '.';
 $string['firstdayofweek'] = '0';
 $string['iso6391'] = 'en';
 $string['iso6392'] = 'eng';
-$string['labelsep'] = '&nbsp;';
+$string['labelsep'] = '';
 $string['listsep'] = ',';
 $string['locale'] = 'en_AU.UTF-8';
 $string['localewin'] = 'English_Australia.1252';
index 0557eb6..cd039f7 100644 (file)
@@ -1373,6 +1373,8 @@ class html_writer {
         $text = trim($text);
         $label = self::tag('label', $text, $attributes);
 
+        /*
+        // TODO $colonize disabled for now yet - see MDL-12192 for details
         if (!empty($text) and $colonize) {
             // the $text may end with the colon already, though it is bad string definition style
             $colon = get_string('labelsep', 'langconfig');
@@ -1386,6 +1388,7 @@ class html_writer {
                 }
             }
         }
+        */
 
         return $label;
     }