Closed
Bug 1263407
Opened 9 years ago
Closed 9 years ago
String with too many spaces
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
INVALID
People
(Reporter: breanajones, Unassigned)
Details
Attachments
(1 file)
781.85 KB,
image/png
|
Details |
This string contains spaces in the source locale and needs to be resolved before translating it into the target locale.
https://mozilla.locamotion.org/es_MX/firefox/translate/browser/chrome/browser/aboutDialog.dtd.po#unit=4081410
Reporter | ||
Updated•9 years ago
|
Component: Translation → General
OS: Unspecified → All
Hardware: Unspecified → All
Summary: String with spaces → String with too many spaces
Version: unspecified → 48 Branch
Reporter | ||
Comment 1•9 years ago
|
||
Sorry - actual location is at mozilla-aurora/browser/chrome/browser/aboutDialog.dtd::community.exp.middle
Comment 2•9 years ago
|
||
https://dxr.mozilla.org/mozilla-aurora/rev/29f1480e9931de5ef94055e97f74411aeddb16ad/browser/locales/en-US/chrome/browser/aboutDialog.dtd#28 has 1 space on either side and looks correct to me.
Can you describe more precisely what the problem is with this string?
Flags: needinfo?(breanajones)
Comment 3•9 years ago
|
||
This string is used in the middle of a sentence, so it has one space before and one after.
https://hg.mozilla.org/releases/mozilla-aurora/file/default/browser/locales/en-US/chrome/browser/aboutDialog.dtd#l28
Strings are allowed to contain trailing or starting spaces for that very reason.
Comment 4•9 years ago
|
||
Breana, if the yellow-red icon is looking scary, it's just because it's hard to show leading and trailing whitespace. It's not that that signals an error in the original string.
Reporter | ||
Comment 5•9 years ago
|
||
My apologies, I had seen the yellow icon before but never with the red line underneath so I assumed it meant an error on your end. I will disregard. Thanks for your help!
Flags: needinfo?(breanajones)
Comment 6•9 years ago
|
||
Cool, resolving per comment 5.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•