Closed
Bug 920009
Opened 11 years ago
Closed 10 years ago
[User Story] Delete multiple contacts
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect, P1)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 956219
People
(Reporter: wmathanaraj, Assigned: arcturus)
References
Details
(Keywords: feature, Whiteboard: [ucid:Comms35, 1.4:P2, ft:comms])
Attachments
(3 files)
1.77 MB,
video/quicktime
|
Details | |
130.81 KB,
image/png
|
Details | |
10.41 KB,
patch
|
Details | Diff | Splinter Review |
User story: As a user I want to be able to select multiple contacts and delete. Acceptance Criteria: AC 1: I am offered in any situation where I can delete a contact a possibility to select multiple contacts
Reporter | ||
Updated•11 years ago
|
Flags: in-moztrap?(jhammink)
Summary: [User Story] Delete multiple contacts → [User Story] Delete multiple contacts (FFOS 1.3)
Assignee | ||
Comment 1•11 years ago
|
||
Hey Ayman, before starting to work on it would like your input. My idea here is reuse the select list that we have for exporting, but instead of that for delete. I'll need some feedback on how the interaction should be. Will prepare a poc so you can take a look. Thanks!
Assignee: nobody → francisco.jordano
Flags: needinfo?(aymanmaat)
Assignee | ||
Comment 2•11 years ago
|
||
Hi folks, prelimiar poc of the multiple delete: https://github.com/arcturus/gaia/tree/bug-920009 (yes perhaps it has a bug, but at least it's trying to show a possible workflow, always pending from Ayman words) Thanks!
Updated•11 years ago
|
Flags: needinfo?(aymanmaat)
Comment 3•11 years ago
|
||
removed the ni? to me by mistake whilst doing some bug admin putting it back.
Flags: needinfo?(aymanmaat)
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #814899 -
Flags: feedback?(aymanmaat)
Comment 5•11 years ago
|
||
Comment on attachment 814899 [details]
poc running on nightly
Hey Francisco
Ok, overall in terms of IxD your totally in the ball park with this. There are just three points that I think need to be tweeked:
1) I know you have used the export contacts as your working template which is correct, however we need to align the visual treatment of this screen to the Delete interface of the message app as they are functionally related (delete should have the same visual treatment and therefore speak the same visual language everywhere) I will ni? Victoria to provide you with the assets.
2) @ 00:21 : When one or more contacts is selected the header should change its label from 'Contacts' to ''n' selected'. This both keeps the user orientated as to the number of contacts selected when the selected contacts are not visible on the screen, therefore lowering their cognitive loading, and also aligns the 'Selection to Delete' functionality to that in the Messages App and Call Log.
3) @ 00:23 : The last part of the alphabet in the column on the right hand side of the screen is covered by the screens Footer and is therefore not accessible. Victoria should be able to fix this when she produces the Visual Design for you.
…Apart from that it all looks good to me
Updated•11 years ago
|
QA Contact: isabelrios
Comment 7•11 years ago
|
||
Hi All, There's no need for Visual Design here as this the deleting action is widely used all across the system. Please use the correct Building Block and you'll have no problem: http://buildingfirefoxos.com/building-blocks/edit-mode.html Thanks.
Flags: needinfo?(vpg)
QA Contact: isabelrios
Comment 8•11 years ago
|
||
Comment 9•11 years ago
|
||
Hey, There's a bug for solving the index length issue when in edit mode: https://bugzilla.mozilla.org/show_bug.cgi?id=923143 I have attached a detailed exam of the layout implementation based on the video shared. A few things are pointed out. If you have any further question or doubt, please needinfo me.
Updated•11 years ago
|
Target Milestone: --- → 1.3 Sprint 3 - 10/25
Reporter | ||
Updated•11 years ago
|
No longer blocks: comms_1.3_committed
Reporter | ||
Updated•11 years ago
|
Blocks: comms_1.3_targeted
Comment 10•11 years ago
|
||
This bug is actively worked but isn't a release blocker, it will land once fixed.
Updated•11 years ago
|
QA Contact: isabelrios
Updated•11 years ago
|
Target Milestone: 1.3 Sprint 3 - 10/25 → ---
Updated•11 years ago
|
blocking-b2g: 1.3? → 1.4?
Updated•11 years ago
|
Summary: [User Story] Delete multiple contacts (FFOS 1.3) → [User Story] Delete multiple contacts
Whiteboard: [ucid:Comms35, 1.4:P2, ft:comms]
Updated•11 years ago
|
Comment 11•11 years ago
|
||
Wilfred, this is not in your excel Google doc comms sheet but it's in your email as targeted. Can you confirm this user story for 1.4 targeted? Thanks
Flags: needinfo?(wmathanaraj)
Reporter | ||
Comment 12•11 years ago
|
||
its in google doc - just not following the other sections - its further down the list. Confimred its target.
Flags: needinfo?(wmathanaraj)
Comment 13•11 years ago
|
||
Can you please provide the patch for the implementation of multiple delete of contacts, as shown in the video ?
Flags: needinfo?(francisco.jordano)
Assignee | ||
Comment 14•11 years ago
|
||
@ashab2g that was a PoC, with some fails, will need to invest some time on it and right now have other committed features. If I find the time in 1.4 will do it. Thanks, F.
Flags: needinfo?(francisco.jordano)
Comment 15•11 years ago
|
||
@francisco, Can you please provide the work in progress patch ? so that we can take it forward. Thanks, Ashay
Flags: needinfo?(francisco.jordano)
Assignee | ||
Comment 16•11 years ago
|
||
Sure the work is in a branch here: https://github.com/arcturus/gaia/tree/bug-920009 It's quite outdated, but easy to follow. Regards, F.
Flags: needinfo?(francisco.jordano)
Comment 17•11 years ago
|
||
Please assign this task to me as I am unable to assign it to myself. Thanks Ashay
Flags: needinfo?(francisco.jordano)
Comment 18•11 years ago
|
||
Hi Francisco, Attaching work in progress patch for Multiple Contact Delete. Please review and let me know your feedback. Regards Ashay
Comment 19•11 years ago
|
||
Hi Wchang, As Francisco is PTO till 11th Jan, Can you please assign some one for patch review and let us know the feedback. Regards Ashay
Flags: needinfo?(wchang)
Comment 20•10 years ago
|
||
Jose, maybe you can help? (In reply to ashayb2g from comment #19) > Hi Wchang, > As Francisco is PTO till 11th Jan, Can you please assign some one for > patch review and let us know the feedback. > > Regards > Ashay
Flags: needinfo?(wchang) → needinfo?(jmcf)
Comment 21•10 years ago
|
||
ashayb2g@gmail.com please could you attach a Github PR in order to properly review your patch?. On the other hand I'm concerned on the changes you have done to navigation.js, are they really necessary? There is code commented as well on your patch ...
Flags: needinfo?(jmcf)
Flags: needinfo?(francisco.jordano)
Comment 22•10 years ago
|
||
@Jose, Please refer https://bugzilla.mozilla.org/show_bug.cgi?id=956219 as an implementation of Multiple Contact Delete. PR is attached in this bug request. Regarding navigation.js, on Multiple Delete screen it was not getting the proper zIndex because of two navigation stack.
Updated•10 years ago
|
Blocks: comms_backlog
Updated•10 years ago
|
Updated•10 years ago
|
Flags: in-moztrap?(jhammink)
You need to log in
before you can comment on or make changes to this bug.
Description
•