1255206 - Change and unify mentions of "tap" to "touch"

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Another one for the unification and content consistency story.

I noticed this in: 

Advanced > Plugins > Tap to play
"Add more search providers" > ...and then tapping the [] icon."
Accessibility > Magnify small areas > "tapping near them"

Not sure if there's anywhere else...

Comment 1

[:Margaret Leibovic]

To fix this bug, you'll need to make a change to strings.xml.in and android_strings.dtd. Here's a good example of a similar patch:
http://hg.mozilla.org/mozilla-central/rev/3ec2d772217d

Comment 2

[Abhishek Nagekar [:Abhi]]

Attachment: /mobile/android/base/locales/en-US/android_strings.dtd

I replaced the occurrences of tap to touch. Is this the appropriate way to solve the issue?

Comment 3

[:Margaret Leibovic]

You should generate a patch for review. To do that, you can follow the directions here:
https://developer.mozilla.org/en-US/docs/Mercurial/Using_Mercurial#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

After that, you can request feedback from a reviewer (me, in this case) using the feedback? flag.

Comment 4

[Dinesh Polathula (:dineshp)]

Attachment: bug1255206.patch

I made the required changes, please verify. I tried looking for other occurences of "tap", but I could not find anything relevant that would need to be changed.

I pasted the output of a simple 'grep' command which I used to search for mentions of "tap" in https://pastebin.mozilla.org/8864712.

If there are any other occurences that I might have overlooked, please let me know.

Comment 5

[:Margaret Leibovic]

Comment on attachment 8733755 [details] [diff] [review]
bug1255206.patch

Review of attachment 8733755 [details] [diff] [review]:
-----------------------------------------------------------------

This looks great, thanks!

Comment 6

[Carsten Book [:Tomcat]]

has problems to apply:

adding 1255206 to series file
renamed 1255206 -> bug1255206.patch
applying bug1255206.patch
patching file mobile/android/base/locales/en-US/android_strings.dtd
Hunk #1 FAILED at 155
1 out of 3 hunks FAILED -- saving rejects to file mobile/android/base/locales/en-US/android_strings.dtd.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working directory
errors during apply, please fix and qrefresh bug1255206.patch

could you take a look, thanks!

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/45dbb2917ee0

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/45dbb2917ee0

Comment 9

[Teodora Vermesan (:TeoVermesan)]

Advanced -> Plugins -> Touch to play
Search -> "Add more search providers" -> ...and then touching the [] icon."
Accessibility -> Magnify small areas -> ...touching near them"
verified as fixed using:
Device: Nexus 7 (Android 5.1.1)
Build: Firefox for Android 48.0a1 (2016-04-05)