871514 - Replace "click" with "touch" on About:firefox update button

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

Description

[Ian Barlow (:ibarlow)]

When tapping the "check for updates button", the string that comes back reads

"Update available; click to download"
or
"Update available; click to install"

We shouldn't be using the word "click" on Android, let's change it to "tap".

Comment 1

[Aaron Train [:aaronmt]]

This was added in bug 827347 because there is no concept of tapping on Ouya.

Comment 2

[Ian Barlow (:ibarlow)]

Ah, I was wondering what brought about the change... Can we special case the Ouya builds, then so that they don't affect devices with touch screens?

Comment 3

[Ian Barlow (:ibarlow)]

Updating title -- for consistency with our updater we should use touch, not tap.

Comment 4

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

I want to give Kats a chance to respond since he championed the change. I do think Ouya has a touchpad on the controller, so "touch" or "tap" might not be as foreign to those users.

Comment 5

[Kartikaya Gupta (email:[e-mail])]

While the Ouya controller does have a touchpad, it is used to control a mouse cursor that appears on the screen, so I think using "click" is the most appropriate for that scenario. Using "tap" might be ok there too, but "touch" doesn't feel right for that scenario. If we absolutely want to use "touch" or "tap" for touch-enabled devices I'm not opposed to special-casing this.

Comment 6

[Ian Barlow (:ibarlow)]

Hm, or we could just change the string everywhere to read "Download update" when one is available

Comment 7

[Kartikaya Gupta (email:[e-mail])]

That sounds good to me. There's another message that shows up after the update is downloaded that says "Updated downloaded; click to install" - I assume we should change that to "Install update" as well.

Comment 8

[Kartikaya Gupta (email:[e-mail])]

Trivial string changes make for good first bugs :) The strings are located in mobile/android/locales/en-US/chrome/about.dtd.

Comment 9

[alexchan7432]

Hi,
I'd like to get involved with FireFox Mobile, figure this would be a good first bug.  Mind if I assign this one to myself?

Comment 10

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

(In reply to alexchan7432 from comment #9)
> Hi,
> I'd like to get involved with FireFox Mobile, figure this would be a good
> first bug.  Mind if I assign this one to myself?

Go ahead. For some information on building Firefox on Android, see:
https://wiki.mozilla.org/Mobile/Fennec/Android

For realtime chatting with other developers, visit #mobile on Mozilla's IRC server:
http://irc.mozilla.org

Comment 11

[Shane Tully (:stully:]

Attachment: change strings as described in comments

Changed updates available and updates downloaded strings as described in comments.

Comment 12

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 752955 [details] [diff] [review]
change strings as described in comments

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

After double-checking the string change procedure [1] (I haven't had to do this very often) it looks like we'll have to update the entity names as well. So for example you can change aboutPage.checkForUpdates.available to aboutPage.checkForUpdates.available2 in the dtd file and update the references to it in about.xhtml as well.

Also please make sure your patch includes a commit message as per [2]

[1] https://developer.mozilla.org/en-US/docs/Making_String_Changes
[2] https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

Comment 13

[Shane Tully (:stully:]

Attachment: 02 Remove "click" from update strings to accomodate touch devices

Change string entity names and reflect changes in about.xhtml (and added commit message; sorry!)

Comment 14

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 752996 [details] [diff] [review]
02 Remove "click" from update strings to accomodate touch devices

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

Looks good, thanks!

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/48b38e667074

Comment 16

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/48b38e667074