619278 - :-moz-ui-invalid should apply as soon as the user tries to submit the form

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[gadjo]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8pre) Gecko/20101214 Firefox/4.0b8pre
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8pre) Gecko/20101214 Firefox/4.0b8pre

Open the URL - it has an text input with "required" and "autofocus" attributes - then press enter. The error message will appear but input should also have red shadow as an invalid field. Shadow appears only after blurring the input

Reproducible: Always

Comment 1

[Mounir Lamouri (:mounir)]

Indeed, when you try to submit the form, you should see the red glow with the error message. This is a minor issue though.

Comment 2

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

I don't really like adding this new UpdateValidityUIBits method but I see no other way to fix that without regressing something else.

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 500806 [details] [diff] [review]
Patch v1

># HG changeset patch
># Parent c09a52a853f032fd372152e4cad4839a86e4c1c4
># User Mounir Lamouri <[e-mail]>
># Date 1294068753 -3600
>
>diff --git a/content/html/content/src/nsHTMLFormElement.cpp b/content/html/content/src/nsHTMLFormElement.cpp
>--- a/content/html/content/src/nsHTMLFormElement.cpp
>+++ b/content/html/content/src/nsHTMLFormElement.cpp
>@@ -1752,16 +1752,24 @@ nsHTMLFormElement::CheckValidFormSubmiss
>            * be notified because we can't know.
>            * Submissions shouldn't happen during parsing so it _should_ be safe.
>            */
> 
>           MOZ_AUTO_DOC_UPDATE(doc, UPDATE_CONTENT_STATE, PR_TRUE);
> 
>           for (PRUint32 i = 0, length = mControls->mElements.Length();
>                i < length; ++i) {
>+            // Input elements can trigger a form submission and we want to
>+            // update the style in that case.
>+            if (mControls->mElements[i]->IsHTML(nsGkAtoms::input) &&
>+                nsContentUtils::IsFocusedContent(mControls->mElements[i])) {
>+              static_cast<nsHTMLInputElement*>(mControls->mElements[i])
>+                ->UpdateValidityUIBits(true);
>+            }

This will mean that calling formelement.checkValidity() will turn on the invalid UI on all invalid controls, right. That's actually sort of cool so I think we should leave that for now. But something that'd be good to get feedback from authors on.

But this function really should be renamed. It's doing a lot more than check validity at this point.

r=me, ideally with the function renamed to something more descriptive.

Comment 4

[Mounir Lamouri (:mounir)]

I think you mixed up nsHTMLFormElement::CheckValidity() and nsHTMLFormElement::CheckValidFormSubmission()... I had to add CheckValidFormSubmission() because the form submit event is sent by the form controls and if the form is invalid, the form submit event should not be sent. It might be something to rewrite for after Gecko 2.0 btw.

Comment 5

[Mounir Lamouri (:mounir)]

Pushed:
http://hg.mozilla.org/mozilla-central/rev/6c8c7ff7505a