From 47661d0d57c6dec8cfacb234590cc51cf7fd5c25 Mon Sep 17 00:00:00 2001 From: NOYB Date: Sun, 10 Jan 2016 00:28:56 -0800 Subject: HTML Compliance - Wizard / pfSense Taffic Shaper / Empty heading.

The align attribute on the img element is obsolete. Use CSS instead. logo-black The border attribute is obsolete. Consider specifying img { border: 0; } in CSS instead. logo-black Bad value FieldValidate(this.value, ^[0-9]+$, ) for attribute onchange on element input: syntax error --- src/usr/local/www/wizard.php | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/usr/local/www/wizard.php b/src/usr/local/www/wizard.php index 9914c5b..6bf58ab 100644 --- a/src/usr/local/www/wizard.php +++ b/src/usr/local/www/wizard.php @@ -215,7 +215,8 @@ do { } while ($oldstepid != $stepid); $pgtitle = array(gettext("Wizard"), gettext($pkg['step'][0]['title'])); //First step is main title of the wizard in the breadcrumb -$pgtitle[] = ($stepid > 0 ? gettext($pkg['step'][$stepid]['title']):''); //Following steps are sub-level breadcrumbs. +$pgtitle[] = ($stepid > 0 ? gettext($pkg['step'][$stepid]['title']):' '); //Following steps are sub-level breadcrumbs. +$shortcut_section = "Wizard"; include("head.inc"); if ($pkg['step'][$stepid]['fields']['field'] != "") { ?> @@ -448,7 +449,7 @@ $completion = ($stepid == 0) ? 0:($stepid * 100) / ($totalsteps -1); ?> -

logo-black



+

logo-black



@@ -536,7 +537,7 @@ if ($pkg['step'][$stepid]['fields']['field'] != "") { 'text', $value ))->setHelp($field['description']) - ->setOnchange(($field['validate']) ? "FieldValidate(this.value, " . $field['validate'] . ", " . $field['message'] . ")":""); + ->setOnchange(($field['validate']) ? "FieldValidate(this.value, \"" . $field['validate'] . "\", \"" . $field['message'] . "\")":""); break; case "text": @@ -557,7 +558,7 @@ if ($pkg['step'][$stepid]['fields']['field'] != "") { $onchange = ""; if ($field['validate']) { - $onchange="FieldValidate(this.value, " . $field['validate'] . ", " . $field['message'] . ")"; + $onchange="FieldValidate(this.value, \"" . $field['validate'] . "\", \"" . $field['message'] . "\")"; } $section->addInput(new Form_Input( @@ -647,7 +648,7 @@ if ($pkg['step'][$stepid]['fields']['field'] != "") { 'password', $value ))->setHelp($field['description']) - ->setOnchange(($field['validate']) ? "FieldValidate(this.value, " . $field['validate'] . ", " . $field['message'] .")":""); + ->setOnchange(($field['validate']) ? "FieldValidate(this.value, \"" . $field['validate'] . "\", \"" . $field['message'] ."\")":""); break; case "certca_selection": @@ -804,7 +805,7 @@ if ($pkg['step'][$stepid]['fields']['field'] != "") { $value ))->setHelp($field['description']) ->setAttribute('rows', $field['rows']) - ->setOnchange(($field['validate']) ? "FieldValidate(this.value, " . $field['validate'] . ", " . $field['message'] . ")":""); + ->setOnchange(($field['validate']) ? "FieldValidate(this.value, \"" . $field['validate'] . "\", \"" . $field['message'] . "\")":""); break; case "submit": -- cgit v1.1