654733 - Stop using 'computer' in offline storage notifications

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

We use the word 'computer' for some of the offline storage notifications: app cache and indexedDB

Let's make it more mobile centric

http://mxr.mozilla.org/mozilla-central/source/mobile/locales/en-US/chrome/browser.properties

Comment 1

[Wesley Johnston (:wesj)]

Attachment: Patch

5 min fix. Is there anything else you know of mfinkle?

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 530054 [details] [diff] [review]
Patch

We need madhava to sign off on this too

Comment 3

[Wesley Johnston (:wesj)]

pushed: http://hg.mozilla.org/mozilla-central/rev/8d0ca70728ff

Comment 4

[Andreea Pod]

Can somebody verify this please?

Comment 5

[Tim Maks van den Broek [:mad_maks]]

note that the l10n community will not notice this change because there is no entity change. if it is important that it also change in other languages who use different words for computer and device you need to change the entity so the l10n dashboard will pick it up. or at least leave a note in the l10n newsgroup

Comment 6

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

http://vingtetun.org/todo will show this.  Also looking at the mobile/locales/en-US/chrome/browser.properties.

Posted message at http://groups.google.com/group/mozilla.dev.l10n/topics

Verified : 
Mozilla/5.0 (Android; Linux armv71; rv6.0a1) Gecko/20110513 Firefox/6.0a1 Fennec/6.0a1
Device: Droid 2 
OS: Android 2.2

Comment 7

[Axel Hecht [:Pike]]

Reopening.

Mark, this patch shouldn't have gotten an r+ from you, honestly.

A note to .l10n isn't the right way forward here. I won't have time to back this out until Monday, so you'll have time to fix the keynames 'til then.

Sorry for the short note, gotta catch a plane in a few.

Comment 8

[Anas Husseini [:linostar]]

As Pike said, this patch lacks 2 things:

1- changing the names of the entities (posting about this in dev-l10n isn't sufficient for various reasons).

2- There are other related uses of the term 'computer' such as in overrides/*. You should probably change those too.

Comment 9

[Wesley Johnston (:wesj)]

(In reply to comment #8)
> 2- There are other related uses of the term 'computer' such as in
> overrides/*. You should probably change those too.

The neterror pages are being redesigned in bug 582048. Similarly the antiphishing ui is being redesigned in bug 651860. The only other references I see are about:home "Tabs from other computers", the password manager (which isn't supported in Fennec and will require changes in toolkit code), and a few various sync strings which shouldn't change:
"From a Firefox Sync connected computer..."
"I'm not near my computer..."
"Please update Firefox on your other computer(s)..."

Changing entity names will require some code changes in ContentPermissionPrompt (or changing the entity names for every single of the "siteWantsTo" entity). Writing some test pages now.

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

patch for entities coming up...

Comment 11

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch - entity and code changes

This patch resets the entity names and makes the non-trivial code changes needed to support those changes.

Let me repeat something I have said in the past: Forcing code changes for string changes is _wrong_! There is no way I could ever feel good about a patch like this. It will require a full QA cycle of functional testing over the affected areas, if those areas can be "well scoped".

Surely there is some way our L10N tools can use something other than entity name to key on changes. Can't we store a hash of the string itself?

Comment 12

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mozilla-central/rev/8e093acb3867

Comment 13

[Aaron Train [:aaronmt]]

Verified fixed
Mozilla/5.0 (Android; Linux armv7l; rv:6.0a1) Gecko/20110518 Firefox/6.0a1 Fennec/6.0a1