Closed
Bug 395590
Opened 17 years ago
Closed 17 years ago
Entity ProtocolFlagError should be rephrased
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla1.9beta2
People
(Reporter: hendrik, Assigned: hendrik)
References
()
Details
Attachments
(1 file)
1.16 KB,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
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•17 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•17 years ago
|
Attachment #280268 -
Flags: review?(peterv)
Comment 2•17 years ago
|
||
You should poke the .l10n newsgroup, though, as many locales do translate "loads" in the "a lot of" meaning. :/
Comment 3•17 years ago
|
||
Posted to .l10n.
Comment 4•17 years ago
|
||
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 5•17 years ago
|
||
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+
Updated•17 years ago
|
Assignee: nobody → hendrik.maryns
Comment 6•17 years ago
|
||
Comment on attachment 280268 [details] [diff] [review] the proposed change Just a wording change.
Attachment #280268 -
Flags: approval1.9?
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Comment 7•17 years ago
|
||
This needs approval before it can be checkin-needed.
Keywords: checkin-needed
Comment 8•17 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 9•17 years ago
|
||
Comment on attachment 280268 [details] [diff] [review] the proposed change a=release drivers
Attachment #280268 -
Flags: approval1.9? → approval1.9+
Updated•17 years ago
|
Keywords: checkin-needed
Comment 10•17 years ago
|
||
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: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M10
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•