Closed
Bug 491739
Opened 15 years ago
Closed 15 years ago
change geolocation prompt strings for 3.5 final
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: beltzner, Assigned: dougt)
Details
(Keywords: fixed1.9.1, late-l10n, Whiteboard: [geo])
Attachments
(3 files, 2 obsolete files)
29.29 KB,
image/png
|
Details | |
4.56 KB,
patch
|
Details | Diff | Splinter Review | |
3.26 KB,
patch
|
Details | Diff | Splinter Review |
Design should look like: foo.com wants to know your location (Learn More...) (Share Location) (Don't Share) [ ] Remember for this site so that's changing 2 strings: geolocation.shareLocation=Share Location geolocation.dontTellThem=Don't Share
Flags: blocking-firefox3.5+
Assignee | ||
Comment 1•15 years ago
|
||
Attachment #376082 -
Flags: review?
Assignee | ||
Comment 2•15 years ago
|
||
Assignee | ||
Updated•15 years ago
|
Attachment #376082 -
Flags: review? → review?(gavin.sharp)
Comment 3•15 years ago
|
||
I'd suggest adding a comment (a localization note) in browser/locales/en-US/chrome/browser/browser.properties that would say something to the effect of: "If you're having trouble with the word Share, please use Allow and Block in your language." The reason for this is that we don't use the verb "Share" almost anywhere in Firefox, so putting it in here requires localizers to come up with a translation of a new word, while Allow and Block are well-established in the l10n land and we use them a lot (especially in privacy-related strings, e.g. in Page Info > Permissions; also see bug 491732) Does this make sense?
Assignee | ||
Updated•15 years ago
|
Attachment #376083 -
Flags: ui-review?(beltzner)
Assignee | ||
Updated•15 years ago
|
Whiteboard: [geo]
Assignee | ||
Comment 4•15 years ago
|
||
Attachment #376082 -
Attachment is obsolete: true
Attachment #376103 -
Flags: review?
Attachment #376082 -
Flags: review?(gavin.sharp)
Comment 5•15 years ago
|
||
Comment on attachment 376103 [details] [diff] [review] patch v.2 I think .accesskey is preferred over just appending "key". We should get an accesskey for the remember checkbox, too (R seems to work), but maybe that's best done in a trunk only followup? Not sure.
Attachment #376103 -
Flags: review? → review+
Comment 6•15 years ago
|
||
I had this applied for testing, figured I'd attach it. We can take this only on trunk if the 1 extra string addition is a problem for branch.
Comment 7•15 years ago
|
||
Attachment #376103 -
Attachment is obsolete: true
Assignee | ||
Comment 8•15 years ago
|
||
gavin, lets do the checkbox access key in bug 491769?
Assignee | ||
Comment 9•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/e4f29e0faf9a http://hg.mozilla.org/releases/mozilla-1.9.1/rev/78d6f3d51d82
Assignee | ||
Comment 10•15 years ago
|
||
Comment on attachment 376083 [details]
screenshot
removing request. this bug has been fixed.
Attachment #376083 -
Flags: ui-review?(beltzner)
You need to log in
before you can comment on or make changes to this bug.
Description
•