190546 - Help: New Mail/News prefs need Help buttons hooked up to correct locations in help content

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[robinf]

Win 2K ns trunk build ID 2003012408:

Steps to reproduce:
1. In Mail/News Preferences, choose either the Windows category or the
Notifications category (these are new)
2. Click the Help button in either panel

Observed: Clicking the Help button opens the Help viewer to the Help Welcome page.

Expected: Clicking the Help button in Windows panel should open help viewer to
"Mail & News Preferences - Windows" section of Help. Clicking the Help button in
Notifications panel should open help viewer to "Mail & News Preferences -
Notifications" section of Help.

Here's how to hook up the Help buttons --

Windows pref panel: use this context info for Help button:
<rdf:li> <rdf:Description ID="mail_prefs_windows" nc:name="Windows"
nc:link="chrome://help/locale/mail_help.html#PREFERENCES_MAILNEWS_WINDOWS"/>
</rdf:li>


Notifications pref panel: use this context info for the Help button:
<rdf:li> <rdf:Description ID="mail_prefs_notifications" nc:name="Notifications"
nc:link="chrome://help/locale/mail_help.html#PREFERENCES_MAILNEWS_NOTIFICATIONS"/>
</rdf:li>

nominating for next release

Comment 1

[Brant Gurganus]

these are documentation issues, not Help browser issues

Comment 2

[robinf]

All placeholders have been checked in for help content in help-toc.rdf and
mail_help.html. Now we need an engineer to update the Help button code. Back to
Seth.

Comment 3

[Samir Gehani]

Mail triage team: nsbeta1+/adt2

Comment 4

[robinf]

*** Bug 196125 has been marked as a duplicate of this bug. ***

Comment 5

[Sean Su]

Attachment: patch v1.0

Comment 6

[Daniel Veditz [:dveditz]]

Comment on attachment 123350 [details] [diff] [review]
patch v1.0

sr=dveditz

Comment 7

[(not reading, please use [e-mail] instead)]

Comment on attachment 123350 [details] [diff] [review]
patch v1.0

r/a=sspitzer

Comment 8

[Sean Su]

patch checked in.  closing bug as fixed.

Comment 9

[sairuh (rarely reading bugmail)]

vrfy'd fixed with 2003.05.15.12.