Closed
Bug 814936
Opened 12 years ago
Closed 10 years ago
[system] use "…" instead of ".." in bluetooth transfer messages
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Firefox OS Graveyard
Gaia::System
Tracking
(Not tracked)
VERIFIED
FIXED
1.4 S4 (28mar)
People
(Reporter: flod, Assigned: flod)
References
Details
Attachments
(1 file)
apps\system\system.properties bluetooth-sendingTitle=Sending Bluetooth transfer.. bluetooth-receivingTitle=Receiving Bluetooth transfer.. bluetooth-sending-progress=Sending Bluetooth transfer.. bluetooth-receiving-progress=Receiving Bluetooth transfer.. What are these ".."? Should it be a single dot "." for title or a "…" for progress? If not, why do we need duplicated strings? kB=kB I think that for English it should be KB, not kB (or at least it's KB everywhere else)
Assignee | ||
Updated•12 years ago
|
Summary: system.properties string review: → system.properties string review: ".." and "kB"
Comment 1•12 years ago
|
||
kB is the correct one, since 'k' symbol (lowercase) stands for kilo-, which is a prefix denoting multiplication of the unit by one thousand. 'K' symbol (uppercase) stands for Kelvin, which is a unit of measurement for temperature.
Comment 2•12 years ago
|
||
by the way, if we're talking about digital information units, which are multiplied by 1024 (not 1000), we should use 'kiB' according to IEC 60027. http://en.wikipedia.org/wiki/Binary_prefix
Assignee | ||
Comment 3•12 years ago
|
||
(In reply to Fernando Pereira Silveira from comment #1) > kB is the correct one, since 'k' symbol (lowercase) stands for kilo-, which > is a prefix denoting multiplication of the unit by one thousand. I kind of remember a bug about this, the point is that (AFAIK) all other products are using KB, not kB. I'm using kB myself for my lcoale, so I agree with you, but the point is also about consistency here. A couple of examples http://mxr.mozilla.org/mozilla-central/source/browser/locales/en-US/chrome/browser/feeds/subscribe.properties#24 http://mxr.mozilla.org/mozilla-central/source/toolkit/locales/en-US/chrome/mozapps/downloads/downloads.properties#63
Comment 4•12 years ago
|
||
I agree with you, consistency is important over all other products, so maybe 'KB' is the right choice, but in an ideal world, it *should* be 'kB' (or even better, 'kiB') I also use 'kB' on pt-BR, which makes me wonder: what Mozilla and localizers think about 'kiB' and binary prefixes?
Assignee | ||
Comment 5•12 years ago
|
||
Found in an old dev-l10n post about Fx 3.1 Bug 106618 and bug 420710
Assignee | ||
Comment 7•11 years ago
|
||
Problem is still present in 1.2
Assignee | ||
Comment 8•10 years ago
|
||
Most locales seem to be already using "…" instead of ".." http://transvision.mozfr.org/string/?entity=apps/system/system.properties:bluetooth-sendingTitle&repo=gaia While KB has been already fixed. I don't think it's useful to use new key IDs at this point, every locale should have its own rules. CCing l10n alias to get a bit more visibility anyway.
Assignee: nobody → francesco.lodolo
Keywords: late-l10n
Summary: system.properties string review: ".." and "kB" → [system] use "…" instead of ".." in bluetooth error messages
Assignee | ||
Comment 9•10 years ago
|
||
I can find the string -progress used and they're pretty easy to spot (just transfer a big file) https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/bluetooth_transfer.js#L336 But I can't find any reference to bluetooth-sendingTitle or bluetooth-receivingTitle, not even in the first landing of code https://github.com/mozilla-b2g/gaia/commit/802bfc770ac1a884947cd93274d0a15457094507#diff-171571dcd7d2b8608173bb1b312b6e4e
Assignee | ||
Updated•10 years ago
|
Summary: [system] use "…" instead of ".." in bluetooth error messages → [system] use "…" instead of ".." in bluetooth transfer messages
Assignee | ||
Comment 10•10 years ago
|
||
Not sure if :ianliu can be a reviewer, flagging him for feedback since it's the main contributor to that file.
Attachment #8393368 -
Flags: review?(timdream)
Attachment #8393368 -
Flags: feedback?(iliu)
Comment 11•10 years ago
|
||
Comment on attachment 8393368 [details] [review] Pull request on Github Thanks.
Attachment #8393368 -
Flags: review?(timdream) → review+
Comment 12•10 years ago
|
||
Comment on attachment 8393368 [details] [review] Pull request on Github Thanks for your help. f+
Attachment #8393368 -
Flags: feedback?(iliu) → feedback+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 13•10 years ago
|
||
Master: https://github.com/mozilla-b2g/gaia/commit/f454a2b1ded4afa8d88a3f782f7ba8a96d12ec10
Status: NEW → RESOLVED
Closed: 10 years ago
status-b2g-v1.4:
--- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.4 S4 (28mar)
Updated•10 years ago
|
status-b2g-v1.4:
fixed → ---
Assignee | ||
Comment 14•10 years ago
|
||
Verified on Keon, master git commit 3977de3c 2014-03-24
Assignee | ||
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•