Confirmed users
821
edits
Gaia/SMS/Scrum/2.2S5: Difference between revisions
→Day 4: 30th January
| Line 194: | Line 194: | ||
=== Day 4: 30th January === | === Day 4: 30th January === | ||
====Steve==== | |||
* Still requesting for the meeting minutes. | |||
* {{Bug|1089154}} - [Messages] investigate scoping CSS rules | |||
** Patch give with some css class removal and refinement, the overall result isn't that impressive(reduce ~40ms in average) but still have some effects. | |||
**: (Julien) could you measure the effect in other apps? | |||
**: (Steve) Not measured yet, but it's good idea. | |||
* {{Bug|1127678}} - [SMS][Text Selection] Selection range is hard to change in SMS | |||
** I think it's not difficult to disable the context menu while in edit mode, but it might need UX comment about whether we should disable the other message's context menu. | |||
* Some investigation about the {{Bug|1125601}}, it seems like a system work to define a better place to enable the pointer-events | |||
Today: | |||
* Investigative some text selection issues | |||
* More testing on {{Bug|1089154}} for other apps used shared styling | |||
* Create RTL refinement for network-alerts | |||
====Julien==== | |||
* {{Bug|1089920}}: fix gaia-header performance | |||
** reviewed the small follow-up: {{Bug|1124582}} -- which also fixes an issue in SMS | |||
** waiting that new gaia-header lands on gaia/master to update the PR for the SMS patch and submit it for review | |||
* {{Bug|1082618}}: add a README file | |||
** no progress | |||
* {{Bug|1122649}}: mozSettings has a synchronous penalty | |||
** not much more move for now. I don't know if we'll be able to make it better. | |||
* still no move for {{Bug|1115796}}'s issue even if Gandalf did measurements that give the same result than mine | |||
* {{Bug|1087981}}: remove the load penalty of notify.js | |||
** landed | |||
* reviewed {{Bug|1124758}}: we don't add the hidden "(+0)" text in single-contact threads now | |||
* spent a lot of time on separate matters and helping contributors | |||
* found {{Bug|1127550}}: looks like we can't send a message while a thread is loading... we can see this on very big threads, but very big threads are the threads we use the most, so it's very visible as a result. | |||
Today: | |||
I want to (again): | |||
* more review | |||
* investigate if read-ahead value changes something in SMS | |||
* move forward some background matters (organize group MMS bugs, and drafts handling issues started some weeks ago) | |||
* land the README file | |||
====Oleg==== | |||
* {{Bug|1122501}} - [Messages][Drafts] Unsaved draft is silently discarded when user tries to forward message | |||
** Updated PR with Steve's advice + added one-line fix for email-group thread (it's impossible to send message to email-contact from Participants view) (in review). | |||
* {{Bug|1124758}} - [Messages] We should not add the hidden (+0) text at the end of the header | |||
** Got r+, landed (landed). | |||
* {{Bug|1104961}} - Intermittent share_activity_test.js | Messages as share target Share via Messages Activity close button Should return to Thread panel if in Participants panel | |||
** Re-run Treeherder the last time with MutationObserver workaround, from ~123 Gij-5 runs, no any shared_activity_test failure, only 23 unrelated. | |||
** Got question from Steve, replying. | |||
Other: | |||
* Asked Ahmed to confirm some RTL issue, before we start work on them. | |||
* Seems I saw bug that prevent us from scrolling long messages in Composer input yesterday night, will try to reproduce again and file bug if it's. | |||
*: (Julien) could be the same than {{Bug|1126777}} ? Just guessing ;) | |||
*: (Oleg) Yeah, I was thinking about it, will check :) | |||
Today: | |||
* Will handle review/feedback/need-info requests; Steve's request in priority. | |||
* Will work on review comments and assigned bugs | |||
====Rishav==== | |||
Today | |||
* Bug :1067275 | |||
** WIP. under review -> julienw | |||
** unit test done. | |||
* Bug: 1037650 | |||
** WIP- > | |||
=== Day 5: 2nd February === | === Day 5: 2nd February === | ||
=== Day 5: 3rd February === | === Day 5: 3rd February === | ||