Closed Bug 201493 Opened 21 years ago Closed 21 years ago

Obsolete help call in Venkman

Categories

(Other Applications Graveyard :: Venkman JS Debugger, defect)

x86
Windows 95
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: neil, Assigned: oeschger)

Details

Attachments

(1 file)

Venkman should be using openHelp() to open help but one of the call sites still
uses toOpenWindowByType.
Rob - How does this do ya? openHelp() comes from
chrome://help/content/contextHelp.js, but you are getting it in the XUL from
globalOverlay.xul or some other higher-up available to venkman.xul.
Comment on attachment 120091 [details] [diff] [review]
updating help viewer command

r=rginda, thanks for the patch :)
Attachment #120091 - Flags: review+
Comment on attachment 120091 [details] [diff] [review]
updating help viewer command

alec--wanna do your help-viewer-regression-fixing-review handiwork on this one
as well? Thanks
Attachment #120091 - Flags: superreview?(alecf)
Status: NEW → ASSIGNED
Comment on attachment 120091 [details] [diff] [review]
updating help viewer command

sr=alecf
Attachment #120091 - Flags: superreview?(alecf) → superreview+
fixed.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Product: Core → Other Applications
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: