Use alternating backgrounds for some of inspector's trees

RESOLVED FIXED

Status

Other Applications
DOM Inspector
--
enhancement
RESOLVED FIXED
14 years ago
11 years ago

People

(Reporter: Christopher Aillon (sabbatical, not receiving bugmail), Assigned: Christopher Aillon (sabbatical, not receiving bugmail))

Tracking

(Blocks: 1 bug)

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: QA Note: refuse to verify until regressions are resolved)

Attachments

(2 attachments)

5.29 KB, patch
Christopher Aillon (sabbatical, not receiving bugmail)
: review+
Christopher Aillon (sabbatical, not receiving bugmail)
: superreview+
Details | Diff | Splinter Review
5.95 KB, patch
Christopher Aillon (sabbatical, not receiving bugmail)
: review+
Christopher Aillon (sabbatical, not receiving bugmail)
: superreview+
Details | Diff | Splinter Review
Comment on attachment 143462 [details] [diff] [review]
Theme updates

Got verbal r=ben,sr=jst
Attachment #143462 - Flags: superreview+
Attachment #143462 - Flags: review+
Comment on attachment 143463 [details] [diff] [review]
Patch to inspector

Verbal r+sr=jst
Attachment #143463 - Flags: superreview+
Attachment #143463 - Flags: review+
Fix checked in 03/09/2004 20:54-20:55 PST.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Comment 6

14 years ago
can this have caused bug 237407 ?
(note: the screenshot there is wrong - it's of an older build)

Updated

14 years ago
Blocks: 237655
This caused bug 237655.  How did a patch that breaks the basic rule of "never
set a background color without setting the foreground color" even get review?

Updated

14 years ago
Whiteboard: QA Note: refuse to verify until regression is resolved

Updated

14 years ago
Blocks: 238498

Updated

14 years ago
No longer blocks: 238498
Whiteboard: QA Note: refuse to verify until regression is resolved → QA Note: refuse to verify until regressions are resolved

Updated

14 years ago
Blocks: 238498
Considering we are frozen for 1.7 and that will be the next stable release,
maybe we should back out both patches for this bug, reopen and retarget for 1.8a.
Product: Core → Other Applications
QA Contact: timeless → dom-inspector
You need to log in before you can comment on or make changes to this bug.