1127722 - [Usage][Text Selection] When first time use Usage app, description on "Data usage limit" page can be selected

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Cost Control, defect)

Description

[Alison Shiue]

Attachment: usage_text_selection_issue.mp4

Build ID               20150129003432
Gaia Revision          6e494f1d2676d231abba7dcc2e2822d1170d2d02
Gaia Date              2015-01-29 06:34:56
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/5e6fac01a72f
Gecko Version          37.0a2
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150129.042943
Firmware Date          Thu Jan 29 04:29:53 EST 2015
Bootloader             L1TC000118D0

STR:
1. Launch Usage app (first use)
2. Click Next -> Next, when at "Data alert" page, click "1 GB" to go into "Data usage limit" page
3. Long press at any word

Expected result:
Nothing happened

Actual result:
Copy and paste function shows, please refer the attachment.

Comment 1

[Salvador de la Puente González [:salva]]

This is intended, see bug 1092964 comment 10.

Comment 2

[Salvador de la Puente González [:salva]]

Wow! I read it "backwards". Sorry. Taking it.

Comment 3

[Salvador de la Puente González [:salva]]

Attachment: Simply adding a missed style for data limits

Marina, do you mind to review the PR, please?

Comment 4

[Marina Rodríguez [:mai]]

Comment on attachment 8557062 [details] [review]
Simply adding a missed style for data limits

LGTM. Works fine.(r+)

Thanks Salva

Comment 5

[Salvador de la Puente González [:salva]]

master: cff3964bebecea1f3292c99f80fff97725c03034

Comment 6

[Salvador de la Puente González [:salva]]

Comment on attachment 8557062 [details] [review]
Simply adding a missed style for data limits

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): incomplete bug 1092964 implementation.
[User impact] if declined: low but inconsistent UX 
[Testing completed]: yes
[Risk to taking this patch] (and alternatives if risky): very low
[String changes made]: nope

Comment 7

[Ryan VanderMeulen [:RyanVM]]

v2.2: https://github.com/mozilla-b2g/gaia/commit/72feae5891a8b5c2fe8b42ee76c10e2e0ca1da09

Comment 8

[Alison Shiue]

Verified on
[2.2]
Gaia-Rev        a52999ce7f783177deb17e267bf003a53e6fde06
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/01446d5231ef
Build-ID        20150205162501
Version         37.0a2
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150205.194212
FW-Date         Thu Feb  5 19:42:24 EST 2015
Bootloader      L1TC000118D0

[3.0]
Gaia-Rev        2b83a6d5d1185a438b5bbd287497ac2743b501db
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/34a66aaaca81
Build-ID        20150205010209
Version         38.0a1
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150205.044014
FW-Date         Thu Feb  5 04:40:24 EST 2015
Bootloader      L1TC000118D0