Closed
Bug 649790
Opened 14 years ago
Closed 13 years ago
[Release Channel] Unlabelled download buttons visible on Firefox/Future Releases download page when multiple locales offered
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
3.0
People
(Reporter: aaronmt, Assigned: rik)
References
()
Details
(Keywords: regression)
Attachments
(3 files, 1 obsolete file)
Currently with Safari Version 5.0.4 (6533.20.27), on the following pages, I am seeing multiple download buttons, when there should only exist a single.
[1] http://www.mozilla.com/en-US/firefox
[2] http://www.mozilla.com/en-US/firefox/channel/
See screenshot attached.
Reporter | ||
Updated•14 years ago
|
OS: Mac OS X → Windows 7
Reporter | ||
Updated•14 years ago
|
OS: Windows 7 → Mac OS X
Reporter | ||
Updated•14 years ago
|
Summary: Multiple download buttons visible on Firefox download page in Safari → Multiple download buttons visible on Firefox/Future Releases download page in Safari
Comment 1•14 years ago
|
||
I am not able to reproduce this on my version of safari
Comment 2•14 years ago
|
||
I'd say this is the symptom of having javascript deactivated as we use javascript to hide/show the download box specific to your OS. If js is deactivated, we show the 3 download boxes. Usually we put a mention in the download boxes to indicate the OS for the download box, since we don't mention it on the channels page, it looks like the same box is repeated 3 times.
Comment 3•14 years ago
|
||
(In reply to comment #2)
> I'd say this is the symptom of having javascript deactivated as we use
> javascript to hide/show the download box specific to your OS. If js is
> deactivated, we show the 3 download boxes. Usually we put a mention in the
> download boxes to indicate the OS for the download box, since we don't mention
> it on the channels page, it looks like the same box is repeated 3 times.
https://www.mozilla.com/en-US/firefox/channel/index.html currently does not show buttons with Javascript disabled. see bug 647678
Reporter | ||
Comment 4•14 years ago
|
||
(In reply to comment #2)
> I'd say this is the symptom of having javascript deactivated as we use
> javascript to hide/show the download box specific to your OS. If js is
> deactivated, we show the 3 download boxes. Usually we put a mention in the
> download boxes to indicate the OS for the download box, since we don't mention
> it on the channels page, it looks like the same box is repeated 3 times.
I have JS enabled in the browser.
Comment 5•14 years ago
|
||
For me this problem is caused by having multiple locales offered for download and therefore happens in other browsers as well (ie. Nightly for me). The buttons don't specify the locale that they link to (like they do on the Firefox landing page).
Steps to reproduce:
1) Change your browser's Accept-Language headers to include en-CA (Canadian English). See Preferences -> Content -> Languages in Firefox.
2) Load https://www.mozilla.com/firefox/channel/
Expected Result:
A labelled button for each locale that is applicable to me
Actual Result:
Unlabelled buttons (which all link to en-US builds for Aurora)
Blocks: 647241
OS: Mac OS X → All
Hardware: x86 → All
Summary: Multiple download buttons visible on Firefox/Future Releases download page in Safari → Unlabelled download buttons visible on Firefox/Future Releases download page when multiple locales offered
Updated•14 years ago
|
Assignee: nobody → steven
Blocks: 646880
Summary: Unlabelled download buttons visible on Firefox/Future Releases download page when multiple locales offered → [Release Channel] Unlabelled download buttons visible on Firefox/Future Releases download page when multiple locales offered
Comment 7•14 years ago
|
||
from bz's comment #17 in bug 650238
Yeah, this is a bug in the page. The relevant code is:
function writeDownloadItems(aProduct)
{
// Show the dynamic links
if (gPlatform == PLATFORM_MAC) {
document.writeln(gDownloadItemMacOS9);
} else if (gPlatform == PLATFORM_OTHER) {
document.writeln(gDownloadItemOtherPlatform);
} else {
var languageIDs = getLanguageIDs(aProduct);
for (var i = 0; i < languageIDs.length; ++i)
writeDownloadItem(languageIDs[i]);
}
}
getLanguageIDs splits your navigator.language ("en-CA" in your case) on '-',
sets "languageCode" to "en", sets "regionCode" to "ca", and then does this:
var currentVersion = gLanguages["en"]["us"][aProduct];
var bestVersion = "";
for (region in gLanguages[languageCode]) {
build = gLanguages[languageCode][region];
if (build[aProduct] && regionCode == region) {
That always tests false for the "fxaurora" product, say, because
gLanguages["en"]["ca"]["fxaurora"] is null. Then we go on to:
// We have a localized build for this language, but not this region.
// Show all available regions and let the user pick.
if (bestVersion != currentVersion) {
For Firefox, I see links to en-US, en-GB, and en-ZA builds as a result (but the
buttons don't say where they're going, of course). But for Aurora, as you
noted they're all en-US. And that's because getDownloadURLForLanguage calls
getDownloadURLForAuroraForLanguage in the "fxaurora" case and that last
function has:
// Force en-US locale for now, when we fix other language replace it with
// "locale"
url += aProduct + "-" + aVersion + ".en-US." + os_file_ext;
So yeah, this page just needs fixing.
Reporter | ||
Comment 8•13 years ago
|
||
This is still an issue. Here is the download page for Firefox 5 via Safari Version 5.0.5 (6533.21.1)
Not seeing this in Chrome 14.0.798.0 (89674)
Attachment #525798 -
Attachment is obsolete: true
Updated•13 years ago
|
Target Milestone: --- → 2.10
Updated•13 years ago
|
Target Milestone: 2.10 → 3.0
Updated•13 years ago
|
Assignee: steven → anthony
Assignee | ||
Comment 10•13 years ago
|
||
Committed r92243 on trunk to fix this. I've added the locale name in the download buttons.
See on https://www-trunk.stage.mozilla.com/en-US/firefox/channel/
Assignee | ||
Comment 11•13 years ago
|
||
Pushed to stage/production with r92273.
Comment 12•13 years ago
|
||
(In reply to comment #11)
> Pushed to stage/production with r92273.
Aaron, please confirm that this is fixed thanks
Reporter | ||
Comment 13•13 years ago
|
||
https://www-trunk.stage.mozilla.com/en-US/firefox/channel/
Reopening as this is not fixed on Safari Version 5.0.5 (6533.21.1)
Reporter | ||
Updated•13 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 14•13 years ago
|
||
This is on purpose. For "en" visitors with no build for their specific locale (en-CA has no specific build), we present the whole set of locales that fit the language.
For a Costa Rican visitor (es-CR), we present the four spanish locales we have.
So this looked fixed for me. If you feel we should do something else, please reopen and assign to :pascalc from web localisation and he'll help decide what is best for everyone.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Comment 15•13 years ago
|
||
qa-verified-trunk https://www-trunk.stage.mozilla.com/en-US/firefox/channel/
Keywords: qawanted
Updated•13 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
Updated•12 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•