All users were logged out of Bugzilla on October 13th, 2018

Entity ProtocolFlagError should be rephrased

RESOLVED FIXED in mozilla1.9beta2

Status

()

RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: hendrik, Assigned: hendrik)

Tracking

Trunk
mozilla1.9beta2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
Created attachment 280268 [details] [diff] [review]
the proposed change

The entity ProtocolFlagError 

ProtocolFlagError = Warning: Protocol handler for '%S' doesn't advertise a security policy.  While loads of such protocols are allowed for now, this is deprecated.  Please see the documentation in nsIProtocolHandler.idl.

in the file /mozilla/dom/locales/en-US/chrome/security/caps.properties is unclear due to the ambiguity of the word ‘loads’.  It could be interpreted as ‘a lot of such protocols are allowed’.

I propose it be changed to ‘... loading of such protocols is allowed...’
Attachment #280268 - Flags: review?(l10n)

Comment 1

11 years ago
Comment on attachment 280268 [details] [diff] [review]
the proposed change

I'm the wrong person to review this. Please look at the cvs history of that file to find a good one.

For reference, I don't think this needs a key change, as it's not semantics. Check out the other English localizations (i.e., en-GB), though.
Attachment #280268 - Flags: review?(l10n)
(Assignee)

Updated

11 years ago
Attachment #280268 - Flags: review?(peterv)
You should poke the .l10n newsgroup, though, as many locales do translate "loads" in the "a lot of" meaning. :/
Posted to .l10n.
Comment on attachment 280268 [details] [diff] [review]
the proposed change

Boris wrote the original and I trust his English better than mine :-).
Attachment #280268 - Flags: review?(peterv) → review?(bzbarsky)
Comment on attachment 280268 [details] [diff] [review]
the proposed change

Makes sense.  It's interesting that people would think of the "loads" colloquialism in this context...  I sure didn't.  ;)
Attachment #280268 - Flags: superreview+
Attachment #280268 - Flags: review?(bzbarsky)
Attachment #280268 - Flags: review+
Assignee: nobody → hendrik.maryns
Comment on attachment 280268 [details] [diff] [review]
the proposed change

Just a wording change.
Attachment #280268 - Flags: approval1.9?
(Assignee)

Updated

11 years ago
Keywords: checkin-needed
This needs approval before it can be checkin-needed.
Keywords: checkin-needed

Comment 8

11 years ago
Just as a warning sign, "just a wording change" doesn't exist.

This is fine because:

- the change clarifies the English sentence without a semantic change
- the problem has been announced in the l10n newsgroup
- I've just CC'ed Mark for en-GB to watch out for this, though I don't think he started on Fx3 yet.
Comment on attachment 280268 [details] [diff] [review]
the proposed change

a=release drivers
Attachment #280268 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
Checking in dom/locales/en-US/chrome/security/caps.properties;
/cvsroot/mozilla/dom/locales/en-US/chrome/security/caps.properties,v  <--  caps.properties
new revision: 1.3; previous revision: 1.2
done
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M10
You need to log in before you can comment on or make changes to this bug.