Closed Bug 894865 Opened 11 years ago Closed 11 years ago

Implement UI for active merge with suggestions from contact detail

Categories

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

All
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:koi+)

RESOLVED FIXED
blocking-b2g koi+

People

(Reporter: crdlc, Assigned: crdlc)

References

Details

(Whiteboard: [u=commsapps-user c=contacts p=7])

Attachments

(1 file)

Implement UI for active merge with suggestions from contact detail
Assignee: nobody → crdlc
Status: NEW → ASSIGNED
Hardware: x86 → All
blocking-b2g: --- → koi?
Depends on: 894334, 889759
Whiteboard: [u=commsapps-user c=contacts p=7]
Attached file Patch v1 with unit testing (deleted) —
Attachment #778352 - Flags: review?(jmcf)
Fixed unit tests and added a new one for the "Find duplicate contacts" button
Blocks: 895797
Added test suite for contacts.MatcherUI(contacts_matcher_ui.js) lib
Attachment #778352 - Attachment description: Patch v1 without unit testing → Patch v1 with unit testing
Comment on attachment 778352 [details] Patch v1 with unit testing pending feedback from Ismael concerning mark-up / CSS. Also there are some comments in Github to be implemented
Attachment #778352 - Flags: review?(jmcf) → review+
Blocks: 896917
Comment on attachment 778352 [details] Patch v1 with unit testing Hola Isma, Could you review the CSS code for the duplicate contacts list? Thanks a lot
Attachment #778352 - Flags: feedback?(igonzaleznicolas)
Comment on attachment 778352 [details] Patch v1 with unit testing Everything looks more/less fine, just a couple of comments in GH Thx!
Attachment #778352 - Flags: feedback?(igonzaleznicolas) → feedback+
Done all comments so I gonna merge
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Blocks: 896994
Depends on: 898390
Blocks: 901444
blocking-b2g: koi? → koi+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: