printForm.php

[ruebenwurzel]

Hallo,

derzeit scheint es einen Mix aus HTML4 konform und HTML5 konform zu geben. Wenn möglich sollte man sich bei der Final für eine Version entscheiden und den Code konsequent anpassen. Aufgrund des vielen HTML-Codes, der nach wie vor noch in diversen Dateien enthalten ist, wird das vermutlich eine Fleisaufgabe werden. Alleine in der printForm.php tummeln sich beide Versionen bunt gemischt über die ganze Datei verstreut.  Da ein Goßteil des dort zu findenden HTML codes HTML4 konform ist, war mein Vorschlag von oben, den Rest auch noch HTML4 konform zu machen. Ich kann aber auch sehr gut mit HTML5 konform leben, wenn es konsequent überall angewendet wird. 

Gruß

[Luisehahne]

Hallo.

Das Thema hatten wir vor mehreren Monaten schon einmal. Kein Problem es HTML4 conform zu machen. HTML5 versteht es ja trotzdem, obwohl besser den DOCTYPE auf HTML5 ändern, alleine wegen der Aufrufe der javascripte und deren Ausführung die für HTML5 optimiert sind und deswegen ein Fallback nicht in Frage kommen dürfte.

Dietmar

[ruebenwurzel]

Hallo,

im Zusammenhang mit meinen persönlichen Anpassungen des Form-Moduls ist mir eine Kleinigkeit aufgefallen, die in der Final Version gefixt werden sollte.

modules/form/printForm.php Zeile 40:

Code Select Expand

$tmp .= '<input type="checkbox" name="field'.$field_id.'" class="checkbtn" id="'.$label_id.'" value="'.$key.'" '.$sChecked.'>'."\n";
und Zeile 57:

Code Select Expand

$tmp .= '<input type="radio" name="field'.$field_id.'" class="radiobtn" id="'.$label_id.'" value="'.$n.'" '.$sChecked.'>'."\n";

sollten geändert werden in:

Code Select Expand

$tmp .= '<input type="checkbox" name="field'.$field_id.'" class="checkbtn" id="'.$label_id.'" value="'.$key.'" '.$sChecked.'/>'."\n";

Code Select Expand

$tmp .= '<input type="radio" name="field'.$field_id.'" class="radiobtn" id="'.$label_id.'" value="'.$n.'" '.$sChecked.'/>'."\n";


es fehlen die / vor dem schließenden >.

Gruß
Matthias