Closed Bug 1080771 Opened 10 years ago Closed 10 years ago

Remove phone number highlighting code

Categories

(Firefox OS Graveyard :: Gaia::Contacts, defect)

defect
Not set
normal

Tracking

(b2g-v2.2 verified)

VERIFIED FIXED
2.1 S7 (24Oct)
Tracking Status
b2g-v2.2 --- verified

People

(Reporter: drs, Assigned: fcampo)

References

Details

(Whiteboard: [p=3])

Attachments

(2 files)

The call info page in the Dialer currently highlights phone numbers when coming from the call log item associated with a contact. In bug 1070050, we decided to remove the highlighting code for the phone number buttons in the Contacts app, since the call info page already has this. The code for this is shared in ContactsButtons, but we should move it to only the dialer, and remove the isMissedCall URL GET parameter from the activity that the Contacts app receives.
Assignee: nobody → jmcf
Target Milestone: --- → 2.1 S7 (Oct24)
Whiteboard: [p=3]
Assignee: jmcf → fernando.campo
Attached file Link to PR (deleted) —
Attachment #8506854 - Flags: review?(jmcf)
Comment on attachment 8506854 [details] Link to PR r=me for the contacts part. requiring review from Doug for the dialer part and the overall verification of the proposed patch thanks Fernando
Attachment #8506854 - Flags: review?(jmcf)
Attachment #8506854 - Flags: review?(drs.bugzilla)
Attachment #8506854 - Flags: review+
Attachment #8506854 - Flags: review?(drs.bugzilla) → review+
Integration tests from verticalhome (not related) failing, I re-launched it but it's taking more time than expected.
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
This bug has been verified as "pass" on latest nightly build of Flame v2.2. STR: 1. There is a contact with several numbers. 2. Make a miss call log using a number of the contact. 3. Long tap the miss call log and go to call information-> contact details Actual results: The highlighting for the phone number button in the Contacts app is removed as expected. See attachment: verified_Flame_v2.2.3gp Reproduce rate: 0/10 Device: Flame v2.2 (Verified) Build ID 20150716162504 Gaia Revision e1e6317f17a840b19af9dbb25f5a771d8d9fa161 Gaia Date 2015-07-15 21:05:11 Gecko Revision https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/418b2388edb0 Gecko Version 37.0 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150716.195810 Firmware Date Thu Jul 16 19:58:22 EDT 2015 Bootloader L1TC000118D0
Status: RESOLVED → VERIFIED
QA Whiteboard: [MGSEI-Triage+]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: