venkman skin/contents.rdf sez 'chrome:displayName="modern"'; should be Modern

RESOLVED FIXED

Status

Other Applications
Venkman JS Debugger
RESOLVED FIXED
16 years ago
13 years ago

People

(Reporter: John Morrison, Assigned: Robert Ginda)

Tracking

Trunk
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

584 bytes, patch
John Morrison
: review+
John Morrison
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

16 years ago
Along the lines of bug 102588 (and leaving aside the question of supplemental 
packages overwriting the author and skin name), I noticed that I had a lower
case value for "modern" in the UI. I believe it comes from this line in 
venkman's contents.rdf.
http://lxr.mozilla.org/seamonkey/source/extensions/venkman/resources/skin/conte
nts.rdf#48
(Assignee)

Comment 1

16 years ago
here comes a patch, shaver/hyatt, r/sr?
Status: NEW → ASSIGNED
(Assignee)

Comment 2

16 years ago
Created attachment 51761 [details] [diff] [review]
patch
(Reporter)

Comment 3

16 years ago
Comment on attachment 51761 [details] [diff] [review]
patch

going out on a limb on this one, but r=jrgm, and take an rs=mozbot, sez me.
Attachment #51761 - Flags: superreview+
Attachment #51761 - Flags: review+
(Assignee)

Comment 4

16 years ago
fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Product: Core → Other Applications
You need to log in before you can comment on or make changes to this bug.