Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Move mailnews start page to chrome://

RESOLVED FIXED in mozilla1.4beta

Status

SeaMonkey
MailNews: Message Display
RESOLVED FIXED
15 years ago
13 years ago

People

(Reporter: Robert Kaiser, Assigned: Robert Kaiser)

Tracking

Trunk
mozilla1.4beta

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

5.69 KB, patch
neil@parkwaycc.co.uk
: review+
(not reading, please use seth@sspitzer.org instead)
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

15 years ago
I (and others) think that's it's bad to load a static page, like the Mozilla
MailNews start page from the web if it could be placed in chrome as well.
The Netscape one's content is dynamic, of course, and therefore it's right to
have it on their website.

This would solve a bunch of issues (along with reducing unneeded traffic on
mozilla.org):
- It would be easy to make it localizable
- It would still be loaded if a users is offline and starts mailnews
- It could be changed to fit specialities of different versions (new functions
that should be listed etc.)
- perhaps a few others

Web traffic can be worth a view here as well, as it seems: I have this page
localized in German Mozilla, and placed it on my own website, and now it causes
over a gig of traffic per month - quite a lot for me, I have to admit...
Because of that, I'll probably place it in chrome anyway in the upcoming
releases, it would be much easier if it would be in chrome:// anyway...

BTW, moving that page into chrome was already discussed a bit in bug 66570, it
seems that issue will need this move as well.
If we do that, we can't replace the mailnews_start_page with a Hammer photo.
(Assignee)

Comment 2

15 years ago
Matti refers to bug 138724 - as this is closed fixed, it's no issue here. :)
(Assignee)

Comment 3

15 years ago
Created attachment 115248 [details] [diff] [review]
patch v1: add start page to en-US.jar, use that one as default

OK, so here's a patch to get this working. This puts the the start page from
http://www.mozilla.org/mailnews/start.html into en-US.jar and changes
region.properties to use this one.
The page is unchanged from the current version, I only did some whitespace
cleanup.
(Assignee)

Comment 4

15 years ago
Taking, as it's my patch.
Assignee: sspitzer → kairo
(Assignee)

Updated

15 years ago
Attachment #115248 - Flags: review?(sspitzer)
Comment on attachment 115248 [details] [diff] [review]
patch v1: add start page to en-US.jar, use that one as default

some of the links in start.html are relative (as the page used to live on
mozilla.org)
Attachment #115248 - Flags: review?(sspitzer) → review-
(Assignee)

Comment 6

15 years ago
Created attachment 115287 [details] [diff] [review]
patch v2: use absolute URLs for links

Uh, well, it's easy to miss that. This patch corrects the links.
Attachment #115248 - Attachment is obsolete: true
(Assignee)

Updated

15 years ago
Attachment #115287 - Flags: review?(sspitzer)

Comment 7

15 years ago
Comment on attachment 115287 [details] [diff] [review]
patch v2: use absolute URLs for links

start.html says mail_mozilla.gif and jar.mn says mozilla_mail.gif :-/
Attachment #115287 - Flags: review?(sspitzer) → review-
(Assignee)

Comment 8

15 years ago
Created attachment 118766 [details] [diff] [review]
patch v3: correct filename reference

Neil:
Yes, I know. I already had corrected jar.mn in my local tree because it
complained at build time. Here's the patch covering that change.
Attachment #115287 - Attachment is obsolete: true
(Assignee)

Updated

15 years ago
Attachment #118766 - Flags: review?(neil)
(Assignee)

Updated

15 years ago
Target Milestone: --- → mozilla1.4beta

Comment 9

15 years ago
Comment on attachment 118766 [details] [diff] [review]
patch v3: correct filename reference

And you updated for bitrot too I see :-)
Attachment #118766 - Flags: superreview?(sspitzer)
Attachment #118766 - Flags: review?(neil)
Attachment #118766 - Flags: review+
(Assignee)

Comment 10

15 years ago
Yes, I have it applied in my local tree and keep it building with trunk, so I
had to update it with jar.mn changes, and a new cvs diff is against current
trunk of course :)
Blocks: 66570
Comment on attachment 118766 [details] [diff] [review]
patch v3: correct filename reference

sr=sspitzer, sorry for the delay, robert.

after this lands, we'll fix the start page (this one is so outdated.)
Attachment #118766 - Flags: superreview?(sspitzer) → superreview+
(Assignee)

Comment 12

15 years ago
RCS file: /cvsroot/mozilla/mailnews/base/resources/locale/en-US/start.html,v
done
Checking in mozilla/mailnews/base/resources/locale/en-US/start.html;
/cvsroot/mozilla/mailnews/base/resources/locale/en-US/start.html,v  <--  start.html
initial revision: 1.1
done
RCS file: /cvsroot/mozilla/mailnews/base/resources/locale/en-US/mail_mozilla.gif,v
done
Checking in mozilla/mailnews/base/resources/locale/en-US/mail_mozilla.gif;
/cvsroot/mozilla/mailnews/base/resources/locale/en-US/mail_mozilla.gif,v  <-- 
mail_mozilla.gif
initial revision: 1.1
done
Checking in mozilla/mailnews/base/resources/locale/en-US/region.properties;
/cvsroot/mozilla/mailnews/base/resources/locale/en-US/region.properties,v  <-- 
region.properties
new revision: 1.4; previous revision: 1.3
done
Checking in mozilla/mailnews/jar.mn;
/cvsroot/mozilla/mailnews/jar.mn,v  <--  jar.mn
new revision: 1.60; previous revision: 1.59
done


OK, checked in. Now let's start making it a really good page :)
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
*** Bug 128785 has been marked as a duplicate of this bug. ***
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.