Confirmed users
291
edits
Gaia/SMS/Scrum/2: Difference between revisions
Jump to navigation
Jump to search
→Day 4: May, 29th
Steveck324 (talk | contribs) (→Oleg) |
Steveck324 (talk | contribs) |
||
| Line 130: | Line 130: | ||
=== Day 4: May, 29th === | === Day 4: May, 29th === | ||
==== Steve ==== | ==== Steve ==== | ||
* Bug 1010093 [Message] Visual refresh for delivery/read report icon in thread view | * {{Bug|1010093}} [Message] Visual refresh for delivery/read report icon in thread view | ||
** Patch landed | ** Patch landed | ||
* Bug 1003820 [Messages] Recipients container grows without limit breaking the layout | * {{Bug|1003820}} [Messages] Recipients container grows without limit breaking the layout | ||
** Still waiting for ui-review again | ** Still waiting for ui-review again | ||
* Bug 963109 - [Messages] Multi-recipient participants view should show back button instead of cross one | * {{Bug|963109}} - [Messages] Multi-recipient participants view should show back button instead of cross one | ||
** Patch given but declined. Need to update for another review | ** Patch given but declined. Need to update for another review | ||
* Bug 1009098 - [Messages] The "composer" is displayed in the "edit" mode | * {{Bug|1009098}} - [Messages] The "composer" is displayed in the "edit" mode | ||
** Just start to work | ** Just start to work | ||
* Clean code reviews: | * Clean code reviews: {{Bug|1014178}} from Doug and {{Bug|963043}} from julien | ||
* [other] | * [other] | ||
* Some discussion with Bug 829820 - Ability to mark selected SMS threads as read | * Some discussion with {{Bug|829820}} - Ability to mark selected SMS threads as read | ||
** There are many feature/recatoring mentioned in the bug. We might need to create a separate bugs for it. | ** There are many feature/recatoring mentioned in the bug. We might need to create a separate bugs for it. | ||
** Have some discussion with UX about the spec. | ** Have some discussion with UX about the spec. | ||
==== Oleg ==== | ==== Oleg ==== | ||
* Bug 1011654 - [B2G][SMS]Opening the messages app after sending a message from the Call log causes the sent notification sound to occur a second time; | * {{Bug|1011654}} - [B2G][SMS]Opening the messages app after sending a message from the Call log causes the sent notification sound to occur a second time; | ||
** Both v1.4 and master patches landed. | ** Both v1.4 and master patches landed. | ||
* Bug 1013295 - [Messages] Migrated messages don't have a proper sentTimestamp | * {{Bug|1013295}} - [Messages] Migrated messages don't have a proper sentTimestamp | ||
** Both v1.4 and master patches landed. Will add related example to our Demo page. | ** Both v1.4 and master patches landed. Will add related example to our Demo page. | ||
* Bug 963013 - [Messages][Refresh] Update bubbles style and layout of the thread. | * {{Bug|963013}} - [Messages][Refresh] Update bubbles style and layout of the thread. | ||
** Made quite big refactoring of message bubble styles (partially to get rid of some magic); | ** Made quite big refactoring of message bubble styles (partially to get rid of some magic); | ||
** Moved message bubble style to a separate file as 200+ related lines deserve its own file :) | ** Moved message bubble style to a separate file as 200+ related lines deserve its own file :) | ||
** Made several rebases as sms.css was updated; | ** Made several rebases as sms.css was updated; | ||
** Will separate carrier header today and ask for code review as it may take time due to refactoring (later today will prepare new screenshots for ui-review). | ** Will separate carrier header today and ask for code review as it may take time due to refactoring (later today will prepare new screenshots for ui-review). | ||
* Bug 1010093 - [Message] Visual refresh for delivery/read report icon in thread view | * {{Bug|1010093}} - [Message] Visual refresh for delivery/read report icon in thread view | ||
** Reviewed. | ** Reviewed. | ||
* Bug 990518 - [Messaging][Visual Refresh] Keyboard suggestions should update messaging composer layout. | * {{Bug|990518}} - [Messaging][Visual Refresh] Keyboard suggestions should update messaging composer layout. | ||
** Would like to test our idea with SettingListener today. | ** Would like to test our idea with SettingListener today. | ||
* [other] | * [other] | ||
Today: will split carrier header in thread view to style it differently, add example to our Demo page regarding to landed "sentTimestamp" patch + will look into | Today: will split carrier header in thread view to style it differently, add example to our Demo page regarding to landed "sentTimestamp" patch + will look into {{Bug|990518}}. | ||
=== Day 5: May, 30th === | === Day 5: May, 30th === | ||