Confirmed users
291
edits
Gaia/SMS/Scrum/2.1S4: Difference between revisions
Jump to navigation
Jump to search
→Day 6: 9th September
Steveck324 (talk | contribs) |
|||
| Line 210: | Line 210: | ||
=== Day 6: 9th September === | === Day 6: 9th September === | ||
====Steve==== | |||
* {{Bug|1048841}} - [Messages] investigate performance hit from IndexedDB at startup (eg: Contact lookup) | |||
** {{Bug|1038176}} launch time latency issue seems able to be closed with gecko patch landed( \o/ ) So I might take some time to finish this nice-to-have item to make the | |||
aggressiveness adjustable by device resolution. | |||
**: (Oleg) Yay! :) | |||
* {{Bug|983172}} - Parsing jpeg header information for downsampling the image for thumbnail | |||
** Patch given and some precious feedback returned. Will find some time to update the patch for review again. | |||
* {{Bug|1058459}} - [SMS] Data shared from other applications are not shown as Drafts until the app is killed and open again | |||
** I've been told that partner are eager to fix this problem in this sprint... Will try out some solution for this. Not sure if it's overkill to use localstorage simply for event bus, but worth to give it a try. | |||
**: (Oleg) Just attached, you can take a look if you have time - simple poc :) Didn't test much, just thrown what I had in mind :) | |||
**: (Steve) Maybe we could ask our partner for testing first. | |||
**: (Oleg) Yeah, and who is contact point from partner there? I'll try to test this patch a bit, ~1h just see major cases. | |||
**: (Steve) sasikala could help or even finish the patch if it works ;) | |||
**: (Oleg) Oh, cool :) It will save some time for us :) Will ask her then <-her Oh, got it :) | |||
Bunch of review | |||
* Eitan a11y's patch r+ and reply some review/ ni?. | |||
Today: | |||
* Update {{Bug|983172}} | |||
* Review deletion patch and call button first | |||
====Oleg==== | |||
* {{Bug|1063043}} - send button doesn't work when keyboard is open | |||
** Asked reporter to try to use send button without swipe navigation and prepared reduced test case for Etienne that demonstrates issue when we tap on button areas close to window edges (awaiting for info from reporter and Etienne). | |||
**: (Steve) So Etienne suspected that this might related to app edge swipe gesture? | |||
**: (Oleg) Nope, that I suspect, didn't get reply from Etienne yet, I've bisected that previously and also see the difference if I turn off swipe navigation... | |||
**: (Steve) ya, just try it out and the edge swipe option really affect the result... But it's nice catch ;) | |||
**: (Oleg) I hope this issue bothers Dietrich, not the other one that we haven't discovered yet :). Btw in test app click target is "Red container", but in our case it "messages-container", weeeeird .. | |||
* {{Bug|1063970}} - [Messages][Refresh] Update styles of phone and url links located inside messages | |||
** While dogfooding noticed that we didn't change styles for the links inside messages, asked Fang on what styles should be changed (awaiting info from VD). | |||
**: (Steve) Thanks for discovering this... Missing this part in the VD spec | |||
**: (Oleg) yeah, my bad, noticed that when was looking for possible cases for "call" action in menu :) | |||
* {{Bug|1058459}} - [SMS] Data shared from other applications are not shown as Drafts until the app is killed and open again | |||
** Prepared small PoC, just was curious if that can work with localstorage as event bus - looks like it works - will attach to the bug and maybe it can give us some hints for better solution. | |||
Other: | |||
* Some reviews, new bug triaging and investigation. | |||
Today: | |||
* Will handle review comments for the patches that currently in review; | |||
* Clean up review queue. | |||
====Julien==== | |||
* PTO until 9/15 | |||
=== Day 7: 10th September === | === Day 7: 10th September === | ||