MDL-12192 temporarily disabled support for colonized labels

The code must be fixed so that the colon is not displayed when the label
is hidden by accessibility CSS.

diff --git a/lang/en/langconfig.php b/lang/en/langconfig.php
index c23bd5c..7905f66 100644 (file)
--- a/lang/en/langconfig.php
+++ b/lang/en/langconfig.php
@@ -29,7 +29,7 @@ $string['decsep'] = '.';
 $string['firstdayofweek'] = '0';
 $string['iso6391'] = 'en';
 $string['iso6392'] = 'eng';
-$string['labelsep'] = ' ';
+$string['labelsep'] = ': ';
 $string['listsep'] = ',';
 $string['locale'] = 'en_AU.UTF-8';
 $string['localewin'] = 'English_Australia.1252';

diff --git a/lib/outputcomponents.php b/lib/outputcomponents.php
index 0557eb6..cd039f7 100644 (file)
--- a/lib/outputcomponents.php
+++ b/lib/outputcomponents.php
@@ -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;
     }