Confirmed users
820
edits
Gaia/SMS/Scrum/2: Difference between revisions
Jump to navigation
Jump to search
→Day 7: June, 3rd
| Line 228: | Line 228: | ||
=== Day 7: June, 3rd === | === Day 7: June, 3rd === | ||
Attending: Steve, Julien, Oleg | |||
====Steve==== | |||
* [http://bugzil.la/1003820 Bug 1003820] [Messages] Recipients container grows without limit breaking the layout | |||
** Patch updated | |||
* [http://bugzil.la/1010690 Bug 1010690] - [Tarako][MMS][Notification] The notification of new MMS does not appear while playing music/video in foreground | |||
** I could only discover utils.js is missing in sms receiving page. But intermittent notification missing issue might still exist after fixing the parsing issue. | |||
* [http://bugzil.la/1009098 Bug 1009098] - [Messages] The "composer" is displayed in the "edit" mode | |||
** Patch landed | |||
* Clean code reviews: [http://bugzil.la/963013 Bug 963013] discussion with Oleg | |||
Today: | |||
* Land [http://bugzil.la/1003820 Bug 1003820] & [http://bugzil.la/1010690 Bug 1010690](partly, will try to reproduce it after landing the patch) | |||
* Start [http://bugzil.la/1017024 bug 1017024] [Messages] [http://bugzil.la/963018 Bug 963018] change panel's default height and leads the notification mispositioned | |||
====Julien==== | |||
So, no patch yet :( | |||
* [http://bugzil.la/1015867 bug 1015867]: max-height for the composer | |||
** still not started yet... | |||
* [http://bugzil.la/990537 Bug 990537] for DSDS refresh. | |||
** I have a working "simulator" now, so I'll start working on this now. The simulator helps me to simulate DSDS on Firefox without changing the code. | |||
(Oleg) -> Yay! How can we play with improved simulator? | |||
* [http://bugzil.la/974867 bug 974867]: send email from MMS: | |||
** there is a new patch ready for review, will review today, didn't look yesterday | |||
* [http://bugzil.la/1007591 bug 1007591]: IAC communication between cost control and SMS, to silence some received SMS from the operator (TEF is doing this patch) | |||
** reviewed it once again, it's moving forward, they want to land it this week too | |||
Today: | |||
* definitely some patches for both my bugs (dsds refresh + composer height) (again) | |||
* some reviews, but I'll try to timebox this. (again) | |||
====Oleg==== | |||
* [http://bugzil.la/963013 Bug 963013] - [Messages][Refresh] Update bubbles style and layout of the thread. | |||
** Discussed "ellipsis vs line wrap" for long subjects with Omega and Jenny and ..... "line wrap" won! So leaving it as it was before (+ adjusted line-height and paddings per Victoria suggestion) :) | |||
** Received all u-review+ I asked (ui-review+, code review in progress) | |||
* [http://bugzil.la/1008890 Bug 1008890] - [Messages] Thread carrier header isn't localized correctly right after language switch | |||
** Got rid of template in favour of direct DOM manipulations as Steve suggested (in review). | |||
* [http://bugzil.la/1017018 Bug 1017018] - [Messages] [http://bugzil.la/963018 Bug 963018] regresses the localizability of the "To:" string in the recipient panel | |||
** Fixed small nit suggested by Steve (reviewed, check-in needed); | |||
** Discussed regression with "word-wrap" with Jenny and Omega - current behaviour looks better for them, so it's not regression anymore, but unexpected small improvement :) | |||
* [http://bugzil.la/925404 Bug 925404] - [B2G] [SMS] Always include the phone number in the SMS Thread UI, even if the carrier is known | |||
** Discussed this bug (including [http://bugzil.la/883911 Bug 883911] - [SMS][MMS] Update all occurrences of "Type ? Carrier ? Number" strings to same format) with Victoria, please see details in [http://bugzil.la/925404 bug 925404] comment 38. | |||
** Started looking into places that should be updated. | |||
=== Day 8: June, 4th === | === Day 8: June, 4th === | ||
=== Day 9: June, 5th === | === Day 9: June, 5th === | ||