Closed Bug 600151 Opened 14 years ago Closed 14 years ago

Remove the :-moz-submit-invalid default style

Categories

(Core :: Layout: Form Controls, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b7

People

(Reporter: mounir, Assigned: mounir)

References

Details

(Keywords: dev-doc-complete, Whiteboard: [good first bug])

Attachments

(1 file)

Currently :-moz-submit-invalid default style is a box shadow. We wont to get ride of that and have no default style. However, authors will still be able to use the pseudo-class.
Summary: Remove the default :-moz-submit-invalid default style → Remove the :-moz-submit-invalid default style
Whiteboard: [good first bug]
Why? When was this decided?
(In reply to comment #1)
> Why? When was this decided?

limi, the UX person working with us on web forms think that the red glow is much more disturbing than helping. So we are going to remove that (probably for beta 8).
Any feedback are welcome.
Attached patch Patch v1Splinter Review
Trivial patch.
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #479719 - Flags: review?(dbaron)
Attachment #479719 - Flags: approval2.0?
Comment on attachment 479719 [details] [diff] [review]
Patch v1

r=dbaron
Attachment #479719 - Flags: review?(dbaron)
Attachment #479719 - Flags: review+
Attachment #479719 - Flags: approval2.0?
Attachment #479719 - Flags: approval2.0+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/20304cf71487
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
So the box-shadow for the actual invalid parts of the form remains, as well as the tooltip when the form is submitted with invalid inputs. Works nicely, IMHO. Are there any docs on a) this pseudo-class and b) how to override the default behavior of invalid forms in author code?
(In reply to comment #6)
> So the box-shadow for the actual invalid parts of the form remains, as well as
> the tooltip when the form is submitted with invalid inputs. Works nicely, IMHO.
> Are there any docs on a) this pseudo-class and b) how to override the default
> behavior of invalid forms in author code?

The only thing that disappear is the box-shadow on the submit buttons when the form is invalid. Everything else remains the same.
a) https://developer.mozilla.org/en/CSS/:-moz-submit-invalid
b) https://developer.mozilla.org/en/CSS/%3ainvalid ("See also" section might have interesting links)
Actually, we need some dev-doc here because we no longer have a default style.
Keywords: dev-doc-needed
(In reply to comment #7)
> The only thing that disappear is the box-shadow on the submit buttons when the
> form is invalid. Everything else remains the same.
> a) https://developer.mozilla.org/en/CSS/:-moz-submit-invalid
> b) https://developer.mozilla.org/en/CSS/%3ainvalid ("See also" section might
> have interesting links)

So the actual error message pop-up is not removable? I couldn't find any documentation to implement custom error handling (although there is the option of changing the error string).
Updated the docs to say that this doesn't get styled by default.
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: