change geolocation prompt strings for 3.5 final

RESOLVED FIXED

Status

()

Firefox
General
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: beltzner, Assigned: dougt)

Tracking

({fixed1.9.1, late-l10n})

3.0 Branch
x86
All
fixed1.9.1, late-l10n
Points:
---
Bug Flags:
blocking-firefox3.5 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [geo])

Attachments

(3 attachments, 2 obsolete attachments)

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

9 years ago
Created attachment 376082 [details] [diff] [review]
patch v.1
Attachment #376082 - Flags: review?
(Assignee)

Comment 2

9 years ago
Created attachment 376083 [details]
screenshot
(Assignee)

Updated

9 years ago
Attachment #376082 - Flags: review? → review?(gavin.sharp)
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

9 years ago
Attachment #376083 - Flags: ui-review?(beltzner)
(Assignee)

Updated

9 years ago
Whiteboard: [geo]
(Assignee)

Comment 4

9 years ago
Created attachment 376103 [details] [diff] [review]
patch v.2
Attachment #376082 - Attachment is obsolete: true
Attachment #376103 - Flags: review?
Attachment #376082 - Flags: review?(gavin.sharp)
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+
Created attachment 376107 [details] [diff] [review]
with Remember accesskey

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.
Created attachment 376108 [details] [diff] [review]
without Remember accesskey
Attachment #376103 - Attachment is obsolete: true
(Assignee)

Comment 8

9 years ago
gavin, lets do the checkbox access key in bug 491769?
(Assignee)

Comment 9

9 years ago
http://hg.mozilla.org/mozilla-central/rev/e4f29e0faf9a
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/78d6f3d51d82
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Keywords: fixed1.9.1
Resolution: --- → FIXED
(Assignee)

Comment 10

9 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.