Closed
Bug 545110
Opened 15 years ago
Closed 14 years ago
Add/port about:support (Troubleshooting Information page) to SeaMonkey
Categories
(SeaMonkey :: General, enhancement)
SeaMonkey
General
Tracking
(blocking-seamonkey2.1 -, seamonkey2.1 wanted)
RESOLVED
FIXED
seamonkey2.1a2
People
(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
Attachments
(2 files, 2 obsolete files)
(deleted),
patch
|
InvisibleSmiley
:
review+
InvisibleSmiley
:
superreview+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
neil
:
review+
kairo
:
feedback+
|
Details | Diff | Splinter Review |
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).
Comment 1•15 years ago
|
||
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
Comment 2•15 years ago
|
||
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.
Assignee | ||
Comment 3•15 years ago
|
||
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.
Assignee | ||
Updated•15 years ago
|
Flags: blocking-seamonkey2.1a1?
Updated•15 years ago
|
Assignee | ||
Comment 4•14 years ago
|
||
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 5•14 years ago
|
||
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
Assignee | ||
Comment 6•14 years ago
|
||
(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 7•14 years ago
|
||
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+
Assignee | ||
Comment 8•14 years ago
|
||
Attachment #445824 -
Attachment is obsolete: true
Attachment #445985 -
Flags: superreview+
Attachment #445985 -
Flags: review+
Assignee | ||
Comment 9•14 years ago
|
||
(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
Assignee | ||
Comment 10•14 years ago
|
||
(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 → ---
Assignee | ||
Comment 11•14 years ago
|
||
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 12•14 years ago
|
||
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+
Updated•14 years ago
|
Attachment #446925 -
Flags: review?(neil) → review+
Assignee | ||
Comment 13•14 years ago
|
||
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]
Assignee | ||
Updated•14 years ago
|
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•