Gaia/SMS/Scrum/1: Difference between revisions

<julienw> I was the last one friday so I get to choose who's starting

<julienw> and that's schung :)

<schung> ok, I still reviewing the navigation patch

<schung> Might need sometime for unit test part

<julienw> don't hesitate to ask here if there's some things not clear

<schung> Sure, it's great without significant bug ;)  

<julienw> :D

<schung> timestamp in bubble is landed with julien's suggestion

<julienw> I'll take screenshots of the timestamp for the sprint page

<schung> And I just take a little time on bug 891344 for low storage issue, I will ping Omega for more UX details

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=891344 nor, --, 2.0 S3 (6june), nobody, NEW, [SMS/MMS] When storage is low with "Device space low" message displayed on status bar, device can't  

<schung> That's all!

<julienw> next is ?

<azasypkin> I can :)

<schung> Oleg?

<azasypkin> I've ensured that format '%A, %B %e' for bug 1001439 is unfortunately changed across different locales so it doesn't make sense to move it to shared locale file. Resolved bug as INVALID;

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1001439 nor, --, ---, nobody, RESO INVALID, [Refactoring] Move '%A, %B %e' date format from app specific files to the shared\locales\date

<azasypkin> Rebased my patch for bug 963013 (wrap date header and message list into single section element) on the latest schung PR, today I'll rebase it on master and ask for review;

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=963013 nor, --, 2.0 S2 (23may), azasypkin, ASSI, [Messages][Refresh] Update bubbles style and layout of the thread.

<julienw> you still need joan's answer though, right ?

<azasypkin> yeah, to be sure that it's the right thing for him

<julienw> oki

<azasypkin> Started reviewing patch for bug 881469;

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=881469 nor, --, 2.0 S2 (23may), felash, NEW, [MMS] Implement Navigation object

<azasypkin> And looked into Composer layout to understand how it works to be better prepared for bug 963018 follow-ups.

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=963018 nor, --, 2.0 S2 (23may), joan.leon, NEW, [Messages][Refresh] Update styles of the Composer

<azasypkin> Today, as I mentioned earlier I'm going to submit for review patch for bug 963013 (and take care of review comments)

<azasypkin> and complete reviewing of navigation object patch.  

<azasypkin> Also once Julien and Joan file JS related follow-ups for bug 963018, I'll will look into it.

<azasypkin> or we already have such bugs filed?

<julienw> nope we don't have it, and I'd like it to be either CSS only, either we don't do it :p

<azasypkin> ah

<azasypkin> anyway we need follow-ups?

<julienw> yes

<azasypkin> ok I meant not only JS only follow-ups then :) That is it from my side

<julienw> okay, I can be next

<arnau> :Julien will you file the follow up bugs?

<julienw> arnau, I can do it, yes

<arnau> If they are css related, you can asign them to me

<arnau> Joan won't be doing follow ups, just land his patch

<julienw> okay, the first thing will be to find a good solution to achieve all requirements

<julienw> I'll post the requirements to the bug

<arnau> ok

<arnau> we could sync offline

<julienw> yes !

<julienw> :)

<julienw> friday I spent a lot of time to review the patch in bug 974867

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=974867 nor, --, ---, na-matsumoto, NEW, [MMS]Auto suggestion for email address

<julienw> they waited for Francisco who's in PTO and I missed their request earlier so it was quite urgent

<julienw> some part of the patch is fine, some other part is not really good, and yet some other parts should be in another patch/bug

<julienw> I spent some time thinking about bug 1001467 and discussing with benoit girard from the graphics team

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1001467 nor, P2, ---, drs+bugzilla, ASSI, [Messages] Background deletion of messages

<julienw> I wonder if a :last-child rule is triggering a style recomputation for each message removal

<julienw> today I'll loo to bug 1011573 that is a blovker

<julienw> blocker

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=1011573 nor, --, ---, nobody, NEW, [B2G][Tarako][Messaging] Keyboard auto closes when editing the 'To' field in the messaging app

<julienw> (or will be, I don't remember)

<julienw> it's a follow up to the simple draft implementation for tarako

<azasypkin> :(

<julienw> and I'll start looking at your comments for the navigation patch

<schung> draft on tarako?

<julienw> schung, yeah, while you were in holidays )

<julienw> arnau, do you hve something SMS-related to add ?

<schung> :(

<julienw> schung, bug 989600

<firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=989600 cri, --, ---, felash, RESO FIXED, [Tarako] Message app MUST save draft when killed by LMK

<julienw> arnau1, do you have something SMS related to add ?

<arnau1> nope, nothing else from my side