Closed Bug 545110 Opened 14 years ago Closed 14 years ago

Add/port about:support (Troubleshooting Information page) to SeaMonkey

Categories

(SeaMonkey :: General, enhancement)

enhancement
Not set
normal

Tracking

(blocking-seamonkey2.1 -, seamonkey2.1 wanted)

RESOLVED FIXED
seamonkey2.1a2
Tracking Status
blocking-seamonkey2.1 --- -
seamonkey2.1 --- wanted

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(2 files, 2 obsolete files)

Firefox implemented about:support in Bug 367596. Bug 518607 aims at moving the basic functionality to Toolkit. This bug is for tracking/implementing the SeaMonkey side, e.g. string additions (en-US l10n).
Should just need three things, a string for the menuitem, a menuitem, and a app.support.baseURL pref, so the "support web site" link goes somewhere other than about:blank.

Well, and it would probably be a good idea to look at what prefs with user values show up, to consider whether anything needs to be added to the whitelist and blacklist.
Depends on: 518607
Bug 548041 is the Thunderbird version from comment0 there:

> 1. port about:support to Thunderbird
> 2. and add smtp, imap and pop settings suitably obscured

#2 is something we should have too.
Depends on: 561668
This is basically working already thanks to the work done by philor. We'll need to port the styles to Modern; filed bug 561668 for that to allow this bug to focus on any non-theme adaptions and customizations.
Flags: blocking-seamonkey2.1a1?
blocking-seamonkey2.1: --- → -
Flags: blocking-seamonkey2.1a1?
Attached patch add menu item (obsolete) — Splinter Review
This is the menu item addition. I left out the onclick=checkForMiddleClick part that FF has for now because we're not using in the other places yet either.

Regarding comment 2 I think that should be a new bug and this one be restricted to adding the menu item.
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #445629 - Flags: superreview?(neil)
Attachment #445629 - Flags: review?(neil)
Comment on attachment 445629 [details] [diff] [review]
add menu item

>     openUILink(formatter.formatURLPref("app.releaseNotesURL"));
>   }
>   catch (ex) { dump(ex); }
> }
> 
>+function goTroubleshootingPage()
>+{
>+  openUILinkIn("about:support", "tab");
Why always in a tab?

>+                oncommand="goTroubleshootingPage()"/>
Nit: missing semicolon
Attached patch menu item v2 (obsolete) — Splinter Review
(In reply to comment #5)
> (From update of attachment 445629 [details] [diff] [review])
> >+function goTroubleshootingPage()
> >+{
> >+  openUILinkIn("about:support", "tab");
> Why always in a tab?

Hmm, I didn't really check whether what FF has made sense, sorry. I used
  goAbout("support");
now, wrapped inside goTroubleshootingPage() for FF compatibility. I can call it directly if you're opposed to that, though.

Before I forget: This probably needs a Help bug.
Attachment #445629 - Attachment is obsolete: true
Attachment #445824 - Flags: superreview?(neil)
Attachment #445824 - Flags: review?(neil)
Attachment #445629 - Flags: superreview?(neil)
Attachment #445629 - Flags: review?(neil)
Comment on attachment 445824 [details] [diff] [review]
menu item v2

>+function goTroubleshootingPage()
Nit: This belongs before goReleaseNotes()
(sorry I didn't notice last time)
Attachment #445824 - Flags: superreview?(neil)
Attachment #445824 - Flags: superreview+
Attachment #445824 - Flags: review?(neil)
Attachment #445824 - Flags: review+
Blocks: 566646
(In reply to comment #2)
> Bug 548041 is the Thunderbird version from comment0 there:
> 
> > 1. port about:support to Thunderbird
> > 2. and add smtp, imap and pop settings suitably obscured
> 
> #2 is something we should have too.

Not too much has happened there. We can either wait and port what they'll come up with or work in parallel. You may file a bug for SM now or later but as I said I think this one is done, thus resolving.

(In reply to comment #6)
> Before I forget: This probably needs a Help bug.

Filed bug 566646.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a2
(In reply to comment #1)
> Should just need three things, a string for the menuitem, a menuitem, and a
> app.support.baseURL pref, so the "support web site" link goes somewhere other
> than about:blank.

Sorry philor, I should have checked your comment more closely. I forgot about the app.support.baseURL part:

Error: formatURLPref: Couldn't get pref: app.support.baseURL
Source File: file:///E:/Programme/seamonkey-trunk/components/nsURLFormatter.js
Line: 68

firefox.js#829:
pref("app.support.baseURL", "http://support.mozilla.com/1/%APP%/%VERSION%/%OS%/%LOCALE%/");

all-thunderbird.js#131:
pref("app.support.baseURL", "http://support.live.mozillamessaging.com/%LOCALE%/%APP%/%APPBUILDID%/");
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Since we have nothing like SUMO or MoMe's support page (localized, per version etc.) I just used our general support page for now. Can be improved later.

KaiRo, what do you think? Would you recommend another page instead?
Attachment #446925 - Flags: review?(neil)
Attachment #446925 - Flags: feedback?(kairo)
Comment on attachment 446925 [details] [diff] [review]
pref patch [Checkin: comment 13]

We unfortunately don't have a SUMO install yet, so I guess this is the best we can do for now, yes.
Attachment #446925 - Flags: feedback?(kairo) → feedback+
Attachment #446925 - Flags: review?(neil) → review+
Comment on attachment 446925 [details] [diff] [review]
pref patch [Checkin: comment 13]

http://hg.mozilla.org/comm-central/rev/22865b405118
Attachment #446925 - Attachment description: pref patch → pref patch [Checkin: comment 13]
Status: REOPENED → RESOLVED
Closed: 14 years ago14 years ago
Resolution: --- → FIXED
Depends on: 605658
You need to log in before you can comment on or make changes to this bug.