1126279 - [Message][Refactoring] Refactor the Delete and markThreadRead methods in thread_list_ui.js

Status: RESOLVED WONTFIX

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

Description

[kumar rishav (:rishav_)]

Refactor the delete and markThreadRead method, and if possible split the delete function. Separate the UI and gecko update method so that it can use effectively in undo action. 
For undo related bugs: Bug: 1121863 and Bug 1037650

Comment 1

[kumar rishav (:rishav_)]

ni? Julien for mentor

Comment 2

[Julien Wajsberg [:julienw]]

You can mark the bugs that need it in the "blocks" line.

What do you expect from me?

Comment 3

[kumar rishav (:rishav_)]

Just a approval to carry work on this as per description or your suggestion on this.
I think no need to blocks for it, i will contniue to work on it once bug 1067275 and bug 1037650 done.

Thanks

Comment 4

[GitHub Autolander Bot]

Attachment: [PullReq] kumarrishav:Bug-1126279 to mozilla-b2g:master

Comment 5

[Julien Wajsberg [:julienw]]

Comment on attachment 8559246 [details] [review]
[PullReq] kumarrishav:Bug-1126279 to mozilla-b2g:master

I added lots of comments.

This is not radically changing what you did; it's mostly cosmetic.

I'm very sorry for the delay. I think I'll be able to get faster now that I have the bug intent in my head. :)

Comment 6

[kumar rishav (:rishav_)]

Yeah, this is mostly cosmetic change, which i did, taking undo action in consideration. Lot's of work is still left in undo patch (though i made it working :) ). If you will say , we can have these changes directly into undo patch and ignore this bug/patch.

Comment 7

[Julien Wajsberg [:julienw]]

I think it's OK to land this first, I prefer several small patches than one big one :)

If it's easier for you to do everything on the undo patch then we can do it like this. This is as you wish.

Comment 8

[Julien Wajsberg [:julienw]]

I left few comments on github but didn't try it yet.

Please put the additional changes in a separate commit :)

Also you might want to wait for bug 1084298 first, it should land tomorrow when Steve wakes up.

Comment 9

[Julien Wajsberg [:julienw]]

Comment on attachment 8559246 [details] [review]
[PullReq] kumarrishav:Bug-1126279 to mozilla-b2g:master

You need a rebase now that bug 1084298 landed.

I added some guidance on github.

Comment 10

[kumar rishav (:rishav_)]

Hi Julien
As lots of changes happened, do you still think it's valid one. Should close it now 
Thanks

Comment 11

[Julien Wajsberg [:julienw]]

mmm I don't think this is obsolete, this is purely a patch for the view, and this didn't change (except it's in conversation.js now).

Comment 12

[kumar rishav (:rishav_)]

assigning to nobody.
Reason : other priority bugs

Comment 13

[Julien Wajsberg [:julienw]]

Mass closing of Gaia::SMS bugs. End of an era :(

Comment 14

[Julien Wajsberg [:julienw]]

Mass closing of Gaia::SMS bugs. End of an era :(