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

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
4 years ago
2 years ago

People

(Reporter: rishav_, Unassigned, Mentored)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
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
(Reporter)

Comment 1

4 years ago
ni? Julien for mentor
Flags: needinfo?(felash)
You can mark the bugs that need it in the "blocks" line.

What do you expect from me?
(Reporter)

Comment 3

4 years ago
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
Assignee: nobody → rishav006
Mentor: felash
Flags: needinfo?(felash)
Created attachment 8559246 [details] [review]
[PullReq] kumarrishav:Bug-1126279 to mozilla-b2g:master
(Reporter)

Updated

4 years ago
Attachment #8559246 - Flags: feedback?(felash)
(Reporter)

Updated

4 years ago
Attachment #8559246 - Flags: feedback?(felash) → review?(felash)
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. :)
Attachment #8559246 - Flags: review?(felash)
(Reporter)

Comment 6

4 years ago
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.
Flags: needinfo?(felash)
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.
Flags: needinfo?(felash)
(Reporter)

Updated

4 years ago
Attachment #8559246 - Flags: review?(felash)
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 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.
Attachment #8559246 - Flags: review?(felash)
Blocks: 992930
OS: Linux → Gonk (Firefox OS)
Hardware: x86_64 → ARM
(Reporter)

Comment 10

3 years ago
Hi Julien
As lots of changes happened, do you still think it's valid one. Should close it now 
Thanks
Flags: needinfo?(felash)
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).
Flags: needinfo?(felash)
(Reporter)

Comment 12

3 years ago
assigning to nobody.
Reason : other priority bugs
Assignee: rishav006 → nobody
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.