Closed Bug 609016 Opened 9 years ago Closed 9 years ago

Remove default style for :invalid

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: mounir, Assigned: mounir)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

With :-moz-ui-invalid introduced, we are going to apply a default style on :-moz-ui-invalid and no longer on :invalid.
Attached patch Patch v1Splinter Review
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #487612 - Flags: review?(dbaron)
Whiteboard: [needs review]
Duplicate of this bug: 609317
Attachment #487612 - Flags: review?(dbaron) → review?(bzbarsky)
Comment on attachment 487612 [details] [diff] [review]
Patch v1

r=me
Attachment #487612 - Flags: review?(bzbarsky) → review+
Dev doc update will be required.
Keywords: dev-doc-needed
Whiteboard: [needs review]
Attachment #487612 - Flags: approval2.0?
Attachment #487612 - Flags: approval2.0? → approval2.0+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/0dccfde3b86c

dev-doc will have to be updated to match bug 609017
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Does this have a net affect on what the user sees? Is :-moz-ui-invalid applied automatically and does it have a default style?
(In reply to comment #6)
> Does this have a net affect on what the user sees? Is :-moz-ui-invalid applied
> automatically and does it have a default style?

:-moz-ui-invalid is a subset of :invalid. IOW, when :-moz-ui-invalid applies, :invalid applies but not the opposite so it will affect what the user see. The dev can still disable :-moz-ui-invalid by disabled :invalid though.
Assigning to me for doc.
Assignee: mounir.lamouri → jswisher
Assignee: jswisher → mounir.lamouri
You need to log in before you can comment on or make changes to this bug.