author | Mike Conley <mconley@mozilla.com> |
Thu, 13 Dec 2012 16:09:54 -0500 | |
changeset 115975 | ea2a1011fdb5262e0e077c7389cd5ffefb3b9904 |
parent 115974 | 79554502913519f7817351088236ffcba99471a3 |
child 115976 | d40e4d1abffd9192c1edf0a61913eb7a23a80284 |
push id | 24034 |
push user | emorley@mozilla.com |
push date | Fri, 14 Dec 2012 15:28:57 +0000 |
treeherder | autoland@50d8f411d305 [default view] [failures only] |
perfherder | [talos] [build metrics] [platform microbench] (compared to previous push) |
reviewers | mak |
bugs | 820533 |
milestone | 20.0a1 |
first release with | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
last release without | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
--- a/browser/components/downloads/content/download.xml +++ b/browser/components/downloads/content/download.xml @@ -28,17 +28,17 @@ because of the downloads summary at the bottom of the list of download items. An element in the summary has the same min-width on a description, and we don't want the panel to change size if the summary isn't being displayed, so we ensure that items share the same minimum width. --> <xul:description class="downloadTarget" crop="center" - style="min-width: &downloadsSummary.minWidth;" + style="min-width: &downloadsSummary.minWidth2;" xbl:inherits="value=target,tooltiptext=target"/> <xul:progressmeter anonid="progressmeter" class="downloadProgress" min="0" max="100" xbl:inherits="mode=progressmode,value=progress"/> <xul:description class="downloadDetails" style="width: &downloadDetails.width;"
--- a/browser/components/downloads/content/downloadsOverlay.xul +++ b/browser/components/downloads/content/downloadsOverlay.xul @@ -111,17 +111,17 @@ align="center" orient="horizontal" onkeypress="DownloadsSummary.onKeyPress(event);" onclick="DownloadsSummary.onClick(event);"> <image class="downloadTypeIcon" /> <vbox> <description id="downloadsSummaryDescription" class="downloadTarget" - style="min-width: &downloadsSummary.minWidth;"/> + style="min-width: &downloadsSummary.minWidth2;"/> <progressmeter id="downloadsSummaryProgress" class="downloadProgress" min="0" max="100" mode="normal" /> <description id="downloadsSummaryDetails" class="downloadDetails" style="width: &downloadDetails.width;"
--- a/browser/components/downloads/src/DownloadsCommon.jsm +++ b/browser/components/downloads/src/DownloadsCommon.jsm @@ -70,17 +70,17 @@ const kDownloadsStringsRequiringFormatti shortTimeLeftHours: true, shortTimeLeftDays: true, statusSeparator: true, statusSeparatorBeforeNumber: true, fileExecutableSecurityWarning: true }; const kDownloadsStringsRequiringPluralForm = { - otherDownloads: true + otherDownloads2: true }; XPCOMUtils.defineLazyGetter(this, "DownloadsLocalFileCtor", function () { return Components.Constructor("@mozilla.org/file/local;1", "nsILocalFile", "initWithPath"); }); const kPartialDownloadSuffix = ".part"; @@ -1855,17 +1855,17 @@ DownloadsSummaryData.prototype = { */ _refreshProperties: function DSD_refreshProperties() { // Pre-load summary with default values. let summary = DownloadsCommon.summarizeDownloads(this._dataItemsForSummary()); this._description = DownloadsCommon.strings - .otherDownloads(summary.numActive); + .otherDownloads2(summary.numActive); this._percentComplete = summary.percentComplete; // If all downloads are paused, show the progress indicator as paused. this._showingProgress = summary.numDownloading > 0 || summary.numPaused > 0; // Display the estimated time left, if present. if (summary.rawTimeLeft == -1) {
--- a/browser/locales/en-US/chrome/browser/downloads/downloads.dtd +++ b/browser/locales/en-US/chrome/browser/downloads/downloads.dtd @@ -20,30 +20,30 @@ For example, in English, a long string would be: 59 minutes, 59 seconds remaining - 1022 of 1023 KB That's 50 characters, so we set the width at 50ch. --> <!ENTITY downloadDetails.width "50ch"> -<!-- LOCALIZATION NOTE (downloadsSummary.minWidth): +<!-- LOCALIZATION NOTE (downloadsSummary.minWidth2): Minimum width for the main description of the downloads summary, which is displayed at the bottom of the Downloads Panel if the number of downloads exceeds the limit that the panel can display. - A good rule of thumb here is to look at the otherDownloads string + A good rule of thumb here is to look at the otherDownloads2 string in downloads.properties, and make a reasonable estimate of its maximum length. For English, this seems like a reasonable limit: - +999 other current downloads + + 999 other downloads - that's 28 characters, so we set the minimum width to 28ch. + that's 21 characters, so we set the minimum width to 21ch. --> -<!ENTITY downloadsSummary.minWidth "28ch"> +<!ENTITY downloadsSummary.minWidth2 "21ch"> <!ENTITY cmd.pause.label "Pause"> <!ENTITY cmd.pause.accesskey "P"> <!ENTITY cmd.resume.label "Resume"> <!ENTITY cmd.resume.accesskey "R"> <!ENTITY cmd.cancel.label "Cancel"> <!ENTITY cmd.cancel.accesskey "C"> <!-- LOCALIZATION NOTE (cmd.show.label, cmd.show.accesskey, cmd.showMac.label,
--- a/browser/locales/en-US/chrome/browser/downloads/downloads.properties +++ b/browser/locales/en-US/chrome/browser/downloads/downloads.properties @@ -66,14 +66,14 @@ shortTimeLeftDays=%1$Sd # even for right-to-left languages, unless the defaults are not suitable. statusSeparator=%1$S \u2014 %2$S statusSeparatorBeforeNumber=%1$S \u2014 %2$S fileExecutableSecurityWarning="%S" is an executable file. Executable files may contain viruses or other malicious code that could harm your computer. Use caution when opening this file. Are you sure you want to launch "%S"? fileExecutableSecurityWarningTitle=Open Executable File? fileExecutableSecurityWarningDontAsk=Don't ask me this again -# LOCALIZATION NOTE (otherDownloads): +# LOCALIZATION NOTE (otherDownloads2): # This is displayed in an item at the bottom of the Downloads Panel when # there are more downloads than can fit in the list in the panel. Use a # semi-colon list of plural forms. # See: http://developer.mozilla.org/en/Localization_and_Plurals -otherDownloads=+%1$S other current download; +%1$S other current downloads +otherDownloads2=+ %1$S other download; + %1$S other downloads