MDL-58179 lesson: Avoid using span with class "label"

This commit is contained in:
Luca Bösch
2018-03-16 08:52:21 +01:00
parent b63a3b04b1
commit ef2b45a977
7 changed files with 42 additions and 42 deletions
+1 -1
View File
@@ -4711,7 +4711,7 @@ abstract class lesson_page extends lesson_base {
$i = 1;
foreach ($answers as $answer) {
$cells = array();
$cells[] = "<span class=\"label\">".get_string("jump", "lesson")." $i<span>: ";
$cells[] = '<label>' . get_string('jump', 'lesson') . ' ' . $i . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);
if ($i === 1){
+2 -2
View File
@@ -234,12 +234,12 @@ class lesson_page_type_branchtable extends lesson_page {
continue;
}
$cells = array();
$cells[] = "<span class=\"label\">".get_string("branch", "lesson")." $i<span>: ";
$cells[] = '<label>' . get_string('branch', 'lesson') . ' ' . $i . '</label>';
$cells[] = format_text($answer->answer, $answer->answerformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("jump", "lesson")." $i<span>: ";
$cells[] = '<label>' . get_string('jump', 'lesson') . ' ' . $i . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);
+11 -11
View File
@@ -258,9 +258,9 @@ class lesson_page_type_matching extends lesson_page {
if ($answer->answer != null) {
$cells = array();
if ($n == 0) {
$cells[] = "<span class=\"label\">".get_string("correctresponse", "lesson").'</span>';
$cells[] = '<label>' . get_string('correctresponse', 'lesson') . '</label>';
} else {
$cells[] = "<span class=\"label\">".get_string("wrongresponse", "lesson").'</span>';
$cells[] = '<label>' . get_string('wrongresponse', 'lesson') . '</label>';
}
$cells[] = format_text($answer->answer, $answer->answerformat, $options);
$table->data[] = new html_table_row($cells);
@@ -268,22 +268,22 @@ class lesson_page_type_matching extends lesson_page {
if ($n == 0) {
$cells = array();
$cells[] = '<span class="label">'.get_string("correctanswerscore", "lesson")."</span>: ";
$cells[] = '<label>' . get_string('correctanswerscore', 'lesson') . '</label>';
$cells[] = $answer->score;
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = '<span class="label">'.get_string("correctanswerjump", "lesson")."</span>: ";
$cells[] = '<label>' . get_string('correctanswerjump', 'lesson') . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);
} elseif ($n == 1) {
$cells = array();
$cells[] = '<span class="label">'.get_string("wronganswerscore", "lesson")."</span>: ";
$cells[] = '<label>' . get_string('wronganswerscore', 'lesson') . '</label>';
$cells[] = $answer->score;
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = '<span class="label">'.get_string("wronganswerjump", "lesson")."</span>: ";
$cells[] = '<label>' . get_string('wronganswerjump', 'lesson') . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);
}
@@ -297,19 +297,19 @@ class lesson_page_type_matching extends lesson_page {
$cells = array();
if ($this->lesson->custom && $answer->score > 0) {
// if the score is > 0, then it is correct
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->custom) {
$cells[] = '<span class="label">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label>' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->jumpto_is_correct($this->properties->id, $answer->jumpto)) {
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else {
$cells[] = '<span class="label">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label>' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
}
$cells[] = format_text($answer->answer, $answer->answerformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = '<span class="label">'.get_string("matchesanswer", "lesson")." $i</span>: ";
$cells[] = '<label>' . get_string('matchesanswer', 'lesson') . ' ' . $i . '</label>';
$cells[] = format_text($answer->response, $answer->responseformat, $options);
$table->data[] = new html_table_row($cells);
}
+7 -7
View File
@@ -277,30 +277,30 @@ class lesson_page_type_multichoice extends lesson_page {
$cells = array();
if ($this->lesson->custom && $answer->score > 0) {
// if the score is > 0, then it is correct
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->custom) {
$cells[] = '<span class="label">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label>' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->jumpto_is_correct($this->properties->id, $answer->jumpto)) {
// underline correct answers
$cells[] = '<span class="correct">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<span class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</span>: \n';
} else {
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
}
$cells[] = format_text($answer->answer, $answer->answerformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("response", "lesson")." $i</span>";
$cells[] = '<label>' . get_string('response', 'lesson') . ' ' . $i . '</label>';
$cells[] = format_text($answer->response, $answer->responseformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("score", "lesson").'</span>';
$cells[] = '<label>' . get_string('score', 'lesson') . '</label>';
$cells[] = $answer->score;
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("jump", "lesson").'</span>';
$cells[] = '<label>' . get_string('jump', 'lesson') . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);
if ($i === 1){
+7 -7
View File
@@ -139,30 +139,30 @@ class lesson_page_type_numerical extends lesson_page {
$cells = array();
if ($this->lesson->custom && $answer->score > 0) {
// if the score is > 0, then it is correct
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . ':</label>';
} else if ($this->lesson->custom) {
$cells[] = '<span class="label">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label>' . get_string('answer', 'lesson') . ' ' . $i . ':</label>';
} else if ($this->lesson->jumpto_is_correct($this->properties->id, $answer->jumpto)) {
// underline correct answers
$cells[] = '<span class="correct">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<span class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</span>: \n';
} else {
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
}
$cells[] = format_text($answer->answer, $answer->answerformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("response", "lesson")." $i</span>";
$cells[] = '<label>' . get_string('response', 'lesson') . ' ' . $i . '</label>';
$cells[] = format_text($answer->response, $answer->responseformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("score", "lesson").'</span>';
$cells[] = '<label>' . get_string('score', 'lesson') . '</label>';
$cells[] = $answer->score;
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("jump", "lesson").'</span>';
$cells[] = '<label>' . get_string('jump', 'lesson') . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);
if ($i === 1){
+7 -7
View File
@@ -199,30 +199,30 @@ class lesson_page_type_shortanswer extends lesson_page {
$cells = array();
if ($this->lesson->custom && $answer->score > 0) {
// if the score is > 0, then it is correct
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->custom) {
$cells[] = '<span class="label">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label>' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->jumpto_is_correct($this->properties->id, $answer->jumpto)) {
// underline correct answers
$cells[] = '<span class="correct">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<span class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</span>: \n';
} else {
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
}
$cells[] = format_text($answer->answer, $answer->answerformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("response", "lesson")." $i</span>";
$cells[] = '<label>' . get_string('response', 'lesson') . ' ' . $i . '</label>';
$cells[] = format_text($answer->response, $answer->responseformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("score", "lesson").'</span>';
$cells[] = '<label>' . get_string('score', 'lesson') . '</label>';
$cells[] = $answer->score;
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("jump", "lesson").'</span>';
$cells[] = '<label>' . get_string('jump', 'lesson') . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);
if ($i === 1){
+7 -7
View File
@@ -124,30 +124,30 @@ class lesson_page_type_truefalse extends lesson_page {
$cells = array();
if ($this->lesson->custom && $answer->score > 0) {
// if the score is > 0, then it is correct
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->custom) {
$cells[] = '<span class="label">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label>' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
} else if ($this->lesson->jumpto_is_correct($this->properties->id, $answer->jumpto)) {
// underline correct answers
$cells[] = '<span class="correct">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<span class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</span>: \n';
} else {
$cells[] = '<span class="labelcorrect">'.get_string("answer", "lesson")." $i</span>: \n";
$cells[] = '<label class="correct">' . get_string('answer', 'lesson') . ' ' . $i . '</label>';
}
$cells[] = format_text($answer->answer, $answer->answerformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("response", "lesson")." $i</span>";
$cells[] = '<label>' . get_string('response', 'lesson') . ' ' . $i . '</label>';
$cells[] = format_text($answer->response, $answer->responseformat, $options);
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("score", "lesson").'</span>';
$cells[] = '<label>' . get_string('score', 'lesson') . '</label>';
$cells[] = $answer->score;
$table->data[] = new html_table_row($cells);
$cells = array();
$cells[] = "<span class=\"label\">".get_string("jump", "lesson").'</span>';
$cells[] = '<label>' . get_string('jump', 'lesson') . '</label>';
$cells[] = $this->get_jump_name($answer->jumpto);
$table->data[] = new html_table_row($cells);