core icon indicating copy to clipboard operation
core copied to clipboard

Add a class to the body

Open bekanntmacher opened this issue 11 years ago • 6 comments

Fügt dem body eine Klasse error hinzu falls das Formular nicht valide ist.

Damit kann zum Beispiel die Formular-Überschrift im Fehlerfall eingefärbt werden, das Formular hervorgehoben werden, etc. In meinem Fall muss ich das Akkordeon-Element, indem sich das Formular befindet, offen halten.

bekanntmacher avatar Sep 09 '14 19:09 bekanntmacher

I like the idea, but maybe form_error would be more specific.

aschempp avatar Sep 10 '14 07:09 aschempp

Das würde ich nicht in den Core einbauen, z.B. kann das ein Durcheinander werden, wenn du mehrere Formulare hast, oder habe ich da was falsch verstanden?

Du kannst auch mit JS eine Klasse error irgendwo hin bringen, wenn es ein error in einem Formular gibt.

Aybee avatar Sep 10 '14 23:09 Aybee

Das würde ich nicht in den Core einbauen

Ich auch nicht unbedingt.

leofeyer avatar Sep 11 '14 10:09 leofeyer

Das nützliche an meinem Vorschlag ist, dass du schon zu Beginn weisst, ah da unten ist ein Fehler und kannst entsprechend reagieren - ohne js! Vor allem bei mobilen Devices ist dies extrem wichtig, denn dort hast du selten das Formular im Viewport.

Das würde ich nicht in den Core einbauen, z.B. kann das ein Durcheinander werden, wenn du mehrere Formalare hast, oder habe ich da was falsch verstanden?

Ich behaupte, dass in über 95% nur ein Formular auf einer Seite ist. Und wenn du mehrere Formulare hast, kannst du immer noch die Logik anpassen.

bekanntmacher avatar Sep 11 '14 11:09 bekanntmacher

Man kann auch eine Klasse form_error_#id# (zusätzlich) machen, dann liesse sich das Problem der mehrfachen Formulare umgehen.=

aschempp avatar Sep 11 '14 16:09 aschempp

Ich hatte jetzt den Fall, dass ein Formular nicht abgesendet werden konnte. Im Qellcode war nichts von "error" zu sehen. Das einzig Sichtbare war der Hinweis im TITLE-Tag https://github.com/contao/core/blob/3.5/system/modules/core/forms/Form.php#L252-L261 Vielleicht wäre da zusätzlich zum TITLE eine Klasse "form_error" im HTML-Tag wirklich nicht schlecht, obwohl dies ja auch wirklich nur was nützt, wenn man z.B.

.form_error form {
  background-color: rgba(255, 0, 0, .2);
}

aufsetzt.

Aybee avatar Jan 19 '18 17:01 Aybee

@bekanntmacher If you still need this, please create the PR again at https://github.com/contao/contao.

leofeyer avatar Oct 17 '22 15:10 leofeyer