MDL-5311 qtype_multichoice: replace label to a for clean button
authorSara Arjona <sara@moodle.com>
Tue, 30 Apr 2019 15:54:23 +0000 (17:54 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Tue, 30 Apr 2019 18:47:33 +0000 (20:47 +0200)
To avoid behat errors with Firefox, the 'Clean my choice' has been
changed from label to link (with role button).

question/type/multichoice/renderer.php
theme/boost/scss/moodle/question.scss
theme/boost/style/moodle.css
theme/classic/style/moodle.css

index 61585b4..1213c8b 100644 (file)
@@ -300,8 +300,8 @@ class qtype_multichoice_single_renderer extends qtype_multichoice_renderer_base
         }
         // Adds an hidden radio that will be checked to give the impression the choice has been cleared.
         $clearchoiceradio = html_writer::empty_tag('input', $clearchoiceradioattrs);
-        $clearchoiceradio .= html_writer::tag('label', get_string('clearchoice', 'qtype_multichoice'),
-            ['for' => $clearchoiceid]);
+        $clearchoiceradio .= html_writer::link('', get_string('clearchoice', 'qtype_multichoice'),
+            ['for' => $clearchoiceid, 'role' => 'button']);
 
         // Now wrap the radio and label inside a div.
         $result = html_writer::tag('div', $clearchoiceradio, ['id' => $clearchoicefieldname, 'class' => $cssclass]);
index 786c7f1..7ece8bb 100644 (file)
@@ -312,7 +312,7 @@ body.path-question-type {
 }
 .qtype_multichoice_clearchoice {
     padding-top: 10px;
-    label {
+    a {
         cursor: pointer;
         text-decoration: underline;
         padding-left: 30px;
index c11e691..304ba37 100644 (file)
@@ -14030,7 +14030,7 @@ body.path-question-type {
 
 .qtype_multichoice_clearchoice {
   padding-top: 10px; }
-  .qtype_multichoice_clearchoice label {
+  .qtype_multichoice_clearchoice a {
     cursor: pointer;
     text-decoration: underline;
     padding-left: 30px; }
index dad63af..6ddf392 100644 (file)
@@ -14281,7 +14281,7 @@ body.path-question-type {
 
 .qtype_multichoice_clearchoice {
   padding-top: 10px; }
-  .qtype_multichoice_clearchoice label {
+  .qtype_multichoice_clearchoice a {
     cursor: pointer;
     text-decoration: underline;
     padding-left: 30px; }