Closed
Bug 1255206
Opened 8 years ago
Closed 8 years ago
Change and unify mentions of "tap" to "touch"
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox48 verified)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | verified |
People
(Reporter: antlam, Assigned: dineshp, Mentored)
Details
(Whiteboard: [good first bug])
Attachments
(1 file, 1 obsolete file)
8.32 KB,
patch
|
Margaret
:
review+
|
Details | Diff | Splinter Review |
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•8 years ago
|
||
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
Mentor: margaret.leibovic
Whiteboard: [good first bug]
Comment 2•8 years ago
|
||
I replaced the occurrences of tap to touch. Is this the appropriate way to solve the issue?
Comment 3•8 years ago
|
||
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.
Assignee | ||
Comment 4•8 years ago
|
||
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.
Attachment #8733755 -
Flags: review?(margaret.leibovic)
Comment 5•8 years ago
|
||
Comment on attachment 8733755 [details] [diff] [review] bug1255206.patch Review of attachment 8733755 [details] [diff] [review]: ----------------------------------------------------------------- This looks great, thanks!
Attachment #8733755 -
Flags: review?(margaret.leibovic) → review+
Updated•8 years ago
|
Assignee: nobody → dpdineshp2
Keywords: checkin-needed
Updated•8 years ago
|
Attachment #8730016 -
Attachment is obsolete: true
Comment 6•8 years ago
|
||
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!
Flags: needinfo?(dpdineshp2)
Updated•8 years ago
|
Flags: needinfo?(dpdineshp2)
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/45dbb2917ee0
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Comment 9•8 years ago
|
||
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)
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•