Closed
Bug 609016
Opened 14 years ago
Closed 14 years ago
Remove default style for :invalid
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla2.0b8
People
(Reporter: mounir, Assigned: mounir)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
1.16 KB,
patch
|
bzbarsky
:
review+
benjamin
:
approval2.0+
|
Details | Diff | Splinter Review |
With :-moz-ui-invalid introduced, we are going to apply a default style on :-moz-ui-invalid and no longer on :invalid.
Assignee | ||
Comment 1•14 years ago
|
||
Assignee | ||
Updated•14 years ago
|
Whiteboard: [needs review]
Assignee | ||
Updated•14 years ago
|
Attachment #487612 -
Flags: review?(dbaron) → review?(bzbarsky)
Comment 3•14 years ago
|
||
Comment on attachment 487612 [details] [diff] [review]
Patch v1
r=me
Attachment #487612 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 4•14 years ago
|
||
Dev doc update will be required.
Keywords: dev-doc-needed
Whiteboard: [needs review]
Assignee | ||
Updated•14 years ago
|
Attachment #487612 -
Flags: approval2.0?
Updated•14 years ago
|
Attachment #487612 -
Flags: approval2.0? → approval2.0+
Assignee | ||
Comment 5•14 years ago
|
||
Pushed:
http://hg.mozilla.org/mozilla-central/rev/0dccfde3b86c
dev-doc will have to be updated to match bug 609017
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Comment 6•14 years ago
|
||
Does this have a net affect on what the user sees? Is :-moz-ui-invalid applied automatically and does it have a default style?
Assignee | ||
Comment 7•14 years ago
|
||
(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.
Comment 9•14 years ago
|
||
Updated doc: https://developer.mozilla.org/en/CSS/:invalid
Keywords: dev-doc-needed → dev-doc-complete
Assignee | ||
Updated•14 years ago
|
Assignee: jswisher → mounir.lamouri
You need to log in
before you can comment on or make changes to this bug.
Description
•