Confirmed users
383
edits
Line 288: | Line 288: | ||
=== Day 7: 30th September === | === Day 7: 30th September === | ||
====Steve==== | |||
* {{Bug|1179628}} - [Messages][NG] Lay out Settings service structure | |||
** Background work, no progress. | |||
* {{Bug|1201016}} - [Messages][NG] Migrate the current Message manager event handling to NGA. | |||
** No progress. | |||
* {{Bug|1207094}} - [Messages] Disable appropriate controls when in low storage condition | |||
** Migrate event handling workaround for devicestorage service(shim). | |||
* {{Bug|1206678}} - Outgoing MMS contains subject that was typed by user, but then hidden | |||
** Created a patch for original behavior and create another bug for the subject removal discussion. | |||
**:(Julien) I left a comment on github ;) Thanks! | |||
* {{Bug|1203886}} - The back button in sms conversation view does not always work | |||
** Verified the notification part and it should be fixed as well. | |||
Today: | |||
* Low storage work. | |||
* Review the patches. | |||
* Working on some blockers if they have any progress. | |||
====Julien==== | |||
Reminder: I'll be at the Paris-Web conference tomorrow and friday. I'll start to handle things during this afternoon so I won't be very much available this afternoon either. | |||
* continued work on {{Bug|1207093}}: finished fixing comments, but I had more questions to Katie before requesting a new review. | |||
* spent a lot of time on bug mail today. Looks like we have a "rich paste" feature that landed and breaks us in a lot of cases... I wish we were notified of such features in advance. This sounds like some not easy work for us to handle :/ See for example {{Bug|1209900}} and {{Bug|1207083}}. | |||
*:(Steve) Maybe I can ask the gecko devs to know if it's possible to disable the rich paste in message app. I don't think we will have sufficient time for handling the image paste feature in 2.5... | |||
*:(Julien) Yeah I asked for it in {{Bug|1207083}} already... I think we could do it if people tell us it's really super important. But then we'll likely get blockers for next 2 months about it. | |||
*: (Steve) BTW the wrong cursor position in {{Bug|1207083}} looks familiar... It seems related to the br element in contenteditable element and the cursor position will be misplaced when the delete the <br> | |||
*: (Oleg) Not sure if it's related, but we have this one as well 1183595, when cursor is misplaced in subject field. | |||
Today: | |||
I want to: | |||
* continue handling review/feedback/NI queue | |||
* continue work on the low storage condition | |||
====Oleg==== | |||
* {{Bug|1176976}} - [Messages][Drafts] Remove the draft saving/replacing action menu | |||
** Tested on device again and could not spot anything wrong, so landed :) (landed) | |||
** Will keep eye on possible regressions. | |||
* {{Bug|1198266}} - [Messages] Use ConversationService in the application | |||
** Got Steve's feedback, waiting for Julien's one :) (awaiting feedback). | |||
* {{Bug|1205874}} - [Messages] After starting a new conversation from previously saved draft, that conversation will be counted twice when using Select All in Inbox. | |||
** No progress (in review). | |||
* {{Bug|1208532}} - [Messages][Refactoring] Extract SystemMessageHandler from ActivityHandler | |||
** Moving forward with this, I hope to have wip for feedback today (in progress). | |||
Other: | |||
* Noticed ready-to-land PR from "{{Bug|1140344}} - [marionette-apps] Support switching between main and inline activity app frames" has been forgotten :) So asked Johan to land this, so it's landed and we have chance to write tests for Drafts shared between two app instances. And other NGA related things as well since we use SharedWorkers, BC. | |||
*: (Steve) That's awesome :) | |||
*: (Oleg) Heh, yeah, but we need to try it out to see if it really works as expected :) I hope so. | |||
* Triaged some draft dialog and auto-draft-saved bugs; | |||
* Tried to help Rishav with the unit tests for his bug. | |||
Today: | |||
* Will handle review/feedback/need-info requests; | |||
* Will work on review comments and assigned bugs. | |||
=== Day 8: 1st October === | === Day 8: 1st October === | ||
=== Day 9: 2nd October === | === Day 9: 2nd October === |