Closed Bug 395590 Opened 18 years ago Closed 18 years ago

Entity ProtocolFlagError should be rephrased

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9beta2

People

(Reporter: hendrik, Assigned: hendrik)

References

()

Details

Attachments

(1 file)

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 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)
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?
Keywords: checkin-needed
This needs approval before it can be checkin-needed.
Keywords: checkin-needed
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
Closed: 18 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M10
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: