Closed
Bug 590877
Opened 14 years ago
Closed 14 years ago
Replace current default page with about:home
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 4.0b7
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta7+ |
People
(Reporter: mak, Assigned: Gavin)
References
Details
(Whiteboard: [about-home])
Attachments
(1 file, 1 obsolete file)
2.39 KB,
patch
|
Details | Diff | Splinter Review |
bug 563723 added about:home, now we have to link it in the product so that it's used instead of the google hosted page.
Reporter | ||
Updated•14 years ago
|
blocking2.0: --- → ?
Reporter | ||
Comment 1•14 years ago
|
||
I guess the magic happens in browserconfig.properties but maybe we want to change something in the process?
cc-ing gavin and mike
Assignee | ||
Comment 2•14 years ago
|
||
We can just change it in browserconfig.properties. Do we want to change it for all branding (nightly, "devpreview", official)?
Reporter | ||
Comment 3•14 years ago
|
||
nightly and unofficial are using other pages from what I see, not the search page
Assignee | ||
Comment 4•14 years ago
|
||
Yeah, I know, but that doesn't mean we can't change them. Given that the branding and search URL replacing works correctly with about:home, I don't think there's any real reason to not switch them. But we can do that in a separate change at some later point, too.
Updated•14 years ago
|
blocking2.0: ? → beta6+
Assignee | ||
Comment 5•14 years ago
|
||
Use about:home as the home page for nightlies, unofficial (alphas/devpreviews), and official branding (betas/releases).
(browser.startup.homepage_reset has been unused since forever - doesn't even have a default in firefox.js anymore)
Attachment #472697 -
Flags: ui-review?(beltzner)
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Updated•14 years ago
|
Whiteboard: [has patch][needs ui-r beltzner]
Comment 6•14 years ago
|
||
Comment on attachment 472697 [details] [diff] [review]
patch
uir+ with the following caveat: we may need to special case zh-CN so that it still points to http://zh-cn.start3.mozilla.com/firefox/ but we can deal with that in a follow-up
Attachment #472697 -
Flags: ui-review?(beltzner) → ui-review+
Assignee | ||
Updated•14 years ago
|
Attachment #472697 -
Flags: review?(dolske)
Updated•14 years ago
|
Whiteboard: [has patch][needs ui-r beltzner] → [has patch]
Comment 7•14 years ago
|
||
Comment on attachment 472697 [details] [diff] [review]
patch
Maybe give the metrics guys a heads up that this change is being made? Shouldn't matter, but just in case.
Attachment #472697 -
Flags: review?(dolske) → review+
Comment 8•14 years ago
|
||
Also, since this is that actual change that moves us away from a Google-hosted start page... I think it would be appropriate for someone (beltzner? shaver?) to formally approve this as being OK wrt to any known contractual obligations. Pretty sure this has already been discussed, but that's 87% assumption!
Comment 9•14 years ago
|
||
(In reply to comment #8)
> Also, since this is that actual change that moves us away from a Google-hosted
> start page... I think it would be appropriate for someone (beltzner? shaver?)
> to formally approve this as being OK wrt to any known contractual obligations.
> Pretty sure this has already been discussed, but that's 87% assumption!
Yeah - we're good.
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 10•14 years ago
|
||
Attachment #472697 -
Attachment is obsolete: true
Comment 11•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [has patch]
Target Milestone: --- → Firefox 4.0b6
Looks like this broke --enable-official-branding:
Preprocessor.Error: ('/home/dbaron/builds/mozilla-central/mozilla/other-licenses/branding/firefox/locales/jar.mn', 3, 'UNDEFINED_VAR', 'AB_CD')
Assignee | ||
Comment 13•14 years ago
|
||
Bustage fix because I failed to test official branding:
https://hg.mozilla.org/mozilla-central/rev/c59be8574f7a
:(
Reporter | ||
Updated•14 years ago
|
Whiteboard: [about-home]
You need to log in
before you can comment on or make changes to this bug.
Description
•