Fennec has a "Preferences" menu, but EVERY other Android app and the OS use the term "Settings". We should probably follow their example.

For comparison, Firefox on uses the term "Options" on Windows and "Preferences" on Mac.

/summon UX

Madhava and I were talking about this last week, and agree that we should call it Settings on Android.

Should we also change "Clear Site Preferences" (in the site options) and "Search Preferences" (in the search field on about:config)?

Yes, please.

While working on this, I found we had a bunch of dead strings that made it hard to find all the places to change. This patch removes them along with some dead files that referenced them.

Comment on attachment 582963 [details] [diff] [review] patch Review of attachment 582963 [details] [diff] [review]: ----------------------------------------------------------------- 1. Why not change gecko_menu.xml's "@+id/preferences" to "@+id/settings", too? You would need to change code references to |R.id.preferences| to |R.id.settings|. 2. Where is the prefsHeaders.label string (from mobile/android/locales/en-US/chrome/preferences.dtd) used? Should its value be changed to "Settings" too?

(In reply to Chris Peterson (:cpeterson) from comment #8) > 1. Why not change gecko_menu.xml's "@+id/preferences" to "@+id/settings", > too? You would need to change code references to |R.id.preferences| to > |R.id.settings|. Sounds like a good idea. Updated. > 2. Where is the prefsHeaders.label string (from > mobile/android/locales/en-US/chrome/preferences.dtd) used? Should its value > be changed to "Settings" too? This is unused (left over from XUL Fennec); I'll post a patch to remove it.

Updated to remove unused strings from preferences.dtd.

Comment on attachment 583013 [details] [diff] [review] clean up unused strings perhaps you could rename preferences.dtd -> aboutHome.dtd? is it used anywhere else?

Comment on attachment 583005 [details] [diff] [review] patch Review of attachment 583005 [details] [diff] [review]: ----------------------------------------------------------------- Looks good.

https://hg.mozilla.org/mozilla-central/rev/05720c79f740 https://hg.mozilla.org/mozilla-central/rev/c7101dec8deb

Retested with: Mozilla/5.0 (Android; Linux armv7l; rv:11.0a1) Gecko/20111221 Firefox/11.0a1 Fennec/11.0a1 HTC Desire Z (Android 2.3) "Settings" string is used in the Fennec Menu. But in about:config still "Search Preferences" and "Add new Preference" strings are used. Per comment4 and comment5 this strings should also be changed. Reopening bug.

Oops, I missed that part. Thanks! Here's the fix.

Follow-up pushed to inbound; this bug can be resolved fixed when this is merged to mozilla-central. https://hg.mozilla.org/integration/mozilla-inbound/rev/04efeb587d1b The main patch landed for Firefox 11, but the follow-up is only in Firefox 12. Since the follow-up only affects about:config, I don't think it's important to land it in Aurora.

Retested with: 12.0a1 (2012-01-10) HTC Desire Z (Android 2.3) Bug no longer reproducible. In about:config Settings string is used.

Verified on Aurora 11.0a2 (2012-01-17): In about:config still "Search Preferences" and "Add new Preference" strings are used.

Carla - I assume you meant to set the flag to "verified"

No, bug is not yet fixed on Aurora. On Nightly instead of "Search Preferences" and "Add new Preference" we use "Search settings" and "Add a new Setting"

Comment on attachment 583532 [details] [diff] [review] patch [Approval Request Comment] This patch did not make it into aurora with the initial merge.

Comment on attachment 583532 [details] [diff] [review] patch [Triage Comment] Mobile only - approved for aurora.

Verified on: Nightly Fennec 12.0a1 (2012-01-19) Samsung Google Nexus S( Android 2.3.6)