Closed Bug 354470 Opened 13 years ago Closed 13 years ago

Add missing line for the security pane to the help code in pageinfo dialog

Categories

(Firefox :: Page Info Window, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 3 alpha5

People

(Reporter: Pike, Assigned: florian)

Details

Attachments

(1 file)

pageinfo.js and xul still try to expose help, likely leftovers from the suite version of it.
But the help pages don't exist, and pressing F1 just errors, as pageinfo.xul doesn't pull contextHelp.js.

We should either make help in page info work again (nah, not really) or just remove the traces, that is, the cmd in pageinfo.xul and doHelpButton in pageinfo.js.
This bug, bug 339102, and bug 314211 should probably be coordinated so they are all in agreement.  
The help page still doesn't exist but that should be fixed in bug 314221.
Now that bug 339102 is fixed, pageInfo.xul pulls contextHelp.js and doHelpButton is nearly correct.
A line for the security panel is missing in helpTopics though. I will attach a patch here for that if it doesn't get fixed in bug 377076.
Assignee: nobody → f.qu
Version: 2.0 Branch → Trunk
Attached patch patchSplinter Review
(In reply to comment #2)
> The help page still doesn't exist but that should be fixed in bug 314221.

I meant in bug 314211 of course.
Attachment #262694 - Flags: review?(mano)
Whiteboard: [checkin needed]
Checking in pageInfo.js;
/cvsroot/mozilla/browser/base/content/pageinfo/pageInfo.js,v  <--  pageInfo.js
new revision: 1.4; previous revision: 1.3
done

resummarized (was: Remove help entry points or hook them up from pageinfo)
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Summary: Remove help entry points or hook them up from pageinfo → Add missing line for the security pane to the help code in pageinfo dialog
Whiteboard: [checkin needed]
Target Milestone: --- → Firefox 3 alpha5
You need to log in before you can comment on or make changes to this bug.