622597 - form.novalidate should not be taken into account for validity UI when focusing an element

Status: RESOLVED FIXED

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

Description

[Mounir Lamouri (:mounir)]

1. Open test case (URL field) ;
2. Type something in the text field ;

Actual result: the text field is invalid but :-moz-ui-invalid doesn't apply.
Expected result: the text field is invalid and :-moz-ui-invalid applies.

(if you un-focus the text field, you will see :-moz-ui-invalid applying)

Comment 1

[Mounir Lamouri (:mounir)]

Wrong testcase...

1. Open test case (URL field) ;
2. Type one character in the text field ;
3. Blur the text field ;
3. Type one character in the text field.

Actual result: the text field is invalid but :-moz-ui-invalid doesn't apply.
Expected result: the text field is invalid and :-moz-ui-invalid applies.

Comment 2

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

Comment 3

[Mounir Lamouri (:mounir)]

Jonas, this is also fixing another issue: if .setCustomValidity() is used and novalidate is set on the form owner, it will prevent the invalid UI to be shown.

Comment 4

[Mounir Lamouri (:mounir)]

Pushed:
http://hg.mozilla.org/mozilla-central/rev/42d7daff9dd3