MDL-30844 Accessibility: orphan label in form action buttons
authorFrederic Massart <fred@moodle.com>
Mon, 28 May 2012 03:05:49 +0000 (11:05 +0800)
committerFrederic Massart <fred@moodle.com>
Mon, 25 Jun 2012 02:18:42 +0000 (10:18 +0800)
lib/form/group.php
lib/form/submit.php
lib/formslib.php
mod/quiz/comment.php

index d8d5ba9..7c734ac 100644 (file)
@@ -90,6 +90,9 @@ class MoodleQuickForm_group extends HTML_QuickForm_group{
                 return 'static';
             }
         } else {
+            if ($this->getGroupType() == 'submit') {
+                return 'actionbuttons';
+            }
             return 'fieldset';
         }
     }
index b4ccb99..af94e4f 100644 (file)
@@ -87,7 +87,7 @@ class MoodleQuickForm_submit extends HTML_QuickForm_submit {
         if ($this->_flagFrozen){
             return 'nodisplay';
         } else {
-            return 'default';
+            return 'actionbuttons';
         }
     }
 
index ac28eb8..eea3ffb 100644 (file)
@@ -2230,6 +2230,8 @@ class MoodleQuickForm_Renderer extends HTML_QuickForm_Renderer_Tableless{
         $this->_elementTemplates = array(
         'default'=>"\n\t\t".'<div id="{id}" class="fitem {advanced}<!-- BEGIN required --> required<!-- END required --> fitem_{type}"><div class="fitemtitle"><label>{label}<!-- BEGIN required -->{req}<!-- END required -->{advancedimg} {help}</label></div><div class="felement {type}<!-- BEGIN error --> error<!-- END error -->"><!-- BEGIN error --><span class="error">{error}</span><br /><!-- END error -->{element}</div></div>',
 
+        'actionbuttons'=>"\n\t\t".'<div id="{id}" class="fitem fitem_actionbuttons fitem_{type}"><div class="felement {type}">{element}</div></div>',
+
         'fieldset'=>"\n\t\t".'<div id="{id}" class="fitem {advanced}<!-- BEGIN required --> required<!-- END required --> fitem_{type}"><div class="fitemtitle"><div class="fgrouplabel"><label>{label}<!-- BEGIN required -->{req}<!-- END required -->{advancedimg} {help}</label></div></div><fieldset class="felement {type}<!-- BEGIN error --> error<!-- END error -->"><!-- BEGIN error --><span class="error">{error}</span><br /><!-- END error -->{element}</fieldset></div>',
 
         'static'=>"\n\t\t".'<div class="fitem {advanced}"><div class="fitemtitle"><div class="fstaticlabel"><label>{label}<!-- BEGIN required -->{req}<!-- END required -->{advancedimg} {help}</label></div></div><div class="felement fstatic <!-- BEGIN error --> error<!-- END error -->"><!-- BEGIN error --><span class="error">{error}</span><br /><!-- END error -->{element}&nbsp;</div></div>',
index 85e6d45..16d3c0e 100644 (file)
@@ -78,11 +78,8 @@ echo $attemptobj->render_question_for_commenting($slot);
 </div>
 <fieldset class="hidden">
     <div>
-        <div class="fitem">
-            <div class="fitemtitle">
-                <div class="fgrouplabel"><label> </label></div>
-            </div>
-            <fieldset class="felement fgroup">
+        <div class="fitem fitem_actionbuttons fitem_fsubmit">
+            <fieldset class="felement fsubmit">
                 <input id="id_submitbutton" type="submit" name="submit" value="<?php
                         print_string('save', 'quiz'); ?>"/>
             </fieldset>