Closed Bug 288173 Opened 20 years ago Closed 18 years ago

remove :-moz-outline from CSS tests

Categories

(Core :: CSS Parsing and Computation, defect)

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

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: annevk, Assigned: dbaron)

Details

Attachments

(1 obsolete file) 

For the upcoming patch I just removed the complete declarations, as they are not
applied now per CSS syntax.
Attached patch patch #1 (obsolete) — — Splinter Review 
This would fix some of the issues. However, you just said on IRC that invalid
tests are perhaps more useful than valid tests with a ;-)...
Attachment #178943 - Flags: review?(bzbarsky)
I'm not likely to get to this any time soon (at least several weeks).
Attachment #178943 - Flags: superreview+
Attachment #178943 - Flags: review?(bzbarsky)
Attachment #178943 - Flags: review+
Is the patch still useful? An lxr search seems to indicate all this code is removed  or at least moved to a different place.
Dunno. If so, feel free to mark it WORKSFORME.
Assignee: annevankesteren+mozilla → dbaron
Comment on attachment 178943 [details] [diff] [review]
patch #1

Patch is at least outdated now.
Attachment #178943 - Attachment is obsolete: true
Doing a search on lxr for :-moz-outline doesn't return anything:
http://lxr.mozilla.org/seamonkey/search?string=%3A-moz-outline
So I'm marking this worksforme.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: