Closed Bug 609017 Opened 9 years ago Closed 9 years ago

Default style for :-moz-ui-invalid

Categories

(Core :: CSS Parsing and Computation, defect)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla2.0b8

People

(Reporter: mounir, Assigned: mounir)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

Patch v1
1.16 KB, patch
bzbarsky
: review+
limi
: ui-review+
sicking
: approval2.0+
Details | Diff | Splinter Review 
Would have to be:
:-moz-ui-invalid {
  box-shadow: 0 0 1.5px 1px red;
}
Attached patch Patch v1 — — Splinter Review 
I'm going to send that to the try server for the UI review.

The UI review should also take into account bug 605124. IOW, the behavior of :-moz-ui-invalid.
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #488188 - Flags: ui-review?(limi)
Attachment #488188 - Flags: review?(bzbarsky)
Keywords: dev-doc-needed
Attachment #488188 - Flags: review?(bzbarsky) → review+
Attachment #488188 - Flags: approval2.0?
The previous will not contain Linux builds (because the try server slaves have old gcc that fail to build something that doesn't even warn with newer gcc...).

All builds can be found at:
http://stage.mozilla.org/pub/mozilla.org/firefox/tryserver-builds/mlamouri@mozilla.com-88e8b1ec3bdb/
Shouldn't this also remove the rule for :invalid in forms.css?
(In reply to comment #4)
> Shouldn't this also remove the rule for :invalid in forms.css?

bug 609016
Attachment #488188 - Flags: ui-review?(limi) → ui-review+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/a3fabf61118e
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
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.