Confirmed users
821
edits
Gaia/SMS/Scrum/2.2S5: Difference between revisions
→Day 5: 4th February
| Line 362: | Line 362: | ||
=== Day 5: 4th February === | === Day 5: 4th February === | ||
====Steve==== | |||
* Fix text selection bugs. Update the selection patch with selectable class. | |||
*: (Julien) There was no answer from George, do you know if we can ask somebody else about the core gecko issue ? | |||
*: (Steve) I think he asked Morris Tseng offline, but I can ni? him on bug. | |||
*: (Julien) maybe; I'd like at least a confirmation they'll work on it in Gecko. I really think it's a better solution because this would make all apps work correctly (including the Web content !!). And then we could move the bug out of the Gaia::SMS component ;) | |||
* {{Bug|1089154}} - [Messages] investigate scoping CSS rules | |||
** Requesting feedback | |||
* {{Bug|1120018}} - [RTL][Messages] Latin characters and numbers are right-aligned on input field | |||
** Provide a patch that only set the dir/unicode-bidi when text content exist, I put more details on bugzilla, but the main reason is the text will still align left after reset the content to empty even dir/unicode-bidi set, and the desktop browser has the same behavior. | |||
**: (Julien) I remember something about the caret issue, I think Oleg could remember if you can ask him, I think we got the issue somewhere else. I think this is related to keeping a <br> after deleting which works around the issue. | |||
**: (Steve) ok, I also vaguely remember some issue about the <br> tag, will confrim with him | |||
* The meeting minutes forwarded. Spent some time on this with Bevis and PM today. | |||
Today: | |||
* Investigative some text selection issues | |||
* RTL refinement for message and network-alerts | |||
* Comments for partner's spec | |||
====Julien==== | |||
* {{Bug|1089920}}: fix gaia-header performance | |||
** {{Bug|1127857}}: UX answered so I'll try to do it today so that we can move forward. | |||
* {{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. | |||
* {{Bug|1125369}}: investigated possible solutions to prevent 2 actions from happening when double-tapping on an icon in the edit_mode. Pavel seemed to like one of my solutions and will help ;) | |||
* discussed with etienne about {{Bug|1117443}} (starting an app/putting an app in background is really slow) | |||
** he's working on some tweaks to make these actions feel better | |||
* {{Bug|1128093}}: sounds like it's an issue in SMS: using pointer-events: none is not a good solution here :/ | |||
*:(Steve) Do we need workaround in app? or wait for gecko fix | |||
*: (Julien) no gecko fix actually, gecko works as expected. It had a bug before and fixed it, and as a result our bug is now showing :D we use pointer-events to prevent clicking on the list, but this is what prevents scrolling as well. So I think we need to not add the click listener instead. | |||
*: (Steve) I see, but why it's not a good solution here? | |||
*: (Julien) because the list needs to be scrolled ? Sorry I don't understand your question :) currently in the app we use "pointer-events: none", this prevents the scroll, so we need to remove it and find another solution to prevent the click. | |||
*: (Steve) Sorry I misunderstood :p . | |||
* {{Bug|1124141}}: no update from gandalf, I'll have a look if I can fix it | |||
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 | |||
* work on blockers | |||
====Teja==== | |||
*{{Bug|961572}} - [Messages][Refactoring] Implement a new independent cancel button for information view's header | |||
** Under review | |||
*{{Bug|986510}} - [Messages app] Replace menu icon with settings icon when no messages in app | |||
** Under review. | |||
*{{Bug|1124624}} - Add transition to the group view panel | |||
** WIP. | |||
=== Day 5: 5th February === | === Day 5: 5th February === | ||
=== Day 5: 6th February === | === Day 5: 6th February === | ||