Closed Bug 236973 Opened 21 years ago Closed 21 years ago

Use alternating backgrounds for some of inspector's trees

Categories

(Other Applications :: DOM Inspector, enhancement)

Product:
Other Applications
Component:
DOM Inspector
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: caillon, Assigned: caillon)

References

Details

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

Attachments

(2 files)

Theme updates
5.29 KB, patch
caillon
: review+
caillon
: superreview+
Details | Diff | Splinter Review
Patch to inspector
5.95 KB, patch
caillon
: review+
caillon
: 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
Closed: 21 years ago
Resolution: --- → FIXED
can this have caused bug 237407 ? (note: the screenshot there is wrong - it's of an older build)
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?
Whiteboard: QA Note: refuse to verify until regression is resolved
Blocks: 238498
No longer blocks: 238498
Whiteboard: QA Note: refuse to verify until regression is resolved → QA Note: refuse to verify until regressions are resolved
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.

Attachment

General

Creator:
Created:
Updated:
Size: