Confirmed users
821
edits
Gaia/SMS/Scrum/2.2S5: Difference between revisions
Jump to navigation
Jump to search
→Day 5: 5th February
| Line 413: | Line 413: | ||
=== Day 5: 5th February === | === Day 5: 5th February === | ||
====Steve==== | |||
* {{Bug|1127678}} - [SMS][Text Selection] Selection range is hard to change in SMS | |||
** Create a small patch to disable the context menu | |||
* {{Bug|1127648}} - [Messages] Attachment can be selected, caret appears | |||
** Create an experiment patch for George, we might able to find some gaia work to avoid this but still have some small bug. | |||
**: (Julien) if user-select is "text" and not "all" we should not be able to select attachments, right ? | |||
**: (Steve) The attachment will not been copied into memory actually, it's just the caret position that "feels" attachment been contained in the selection. | |||
**: (Julien) ah ok... If it's a feeling, maybe UX can help deciding what we should do? | |||
* {{Bug|1089154}} - [Messages] investigate scoping CSS rules | |||
** Seems Oleg still unable to saw any improvement at his side, will retry on current master. | |||
**: (Julien) tell me if you want that I have a look as well. Note: I've noticed we have a ".XXX *" selector somewhere too. | |||
**: (Steve) Definitely great if you have some free time for testing this patch :) | |||
* {{Bug|1120018}} - [RTL][Messages] Latin characters and numbers are right-aligned on input field | |||
** Found the bug created by Oleg in platform. We could land this in v2.2 or even on master and remove the workaround once the platform bug fixed. | |||
* The meeting minutes replied to PM. Just ping me if you're interested in the answers for spec. | |||
Today: | |||
* Investigative some text selection issues | |||
* Land RTL patch | |||
====Julien==== | |||
* {{Bug|1089920}}: fix gaia-header performance | |||
** {{Bug|1127857}}: r+ patch but I need to discuss a nit with Wilson; hopefully will land today, in that case I'll land the full new gaia-header to master and I'll be able to prepare a patch for {{Bug|1089920}} :) | |||
** If I have time I want to prepare a patch to remove the textContent of all gaia-headers in gaia from the index.html files, because this makes fontFit run twice: once before localization, once after localization. | |||
* {{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 ;) | |||
** Still need to discuss with Pavel on IRC about this | |||
* {{Bug|1128093}}: sounds like it's an issue in SMS: using pointer-events: none is not a good solution here :/ | |||
** Propose an idea but then I thought it was not right. 2 possible solutions: either point 'pointer-events: none' on the A elements, or have a new flavor for ContactRenderer with another template, that would not use the A elements. The second solution is cleaner but also longer :) I'll try to do it today. | |||
* {{Bug|1124141}}: langpacks breaking SMS in Nightly + running unit tests that needs l10n.js in nightly | |||
** proposed a patch, it's been r+; will likely land today as well | |||
**(Steve): Great, I hope this could land ASAP :D | |||
* reviewed Steve's patches about user selection and RTL in the input composer. | |||
Today: | |||
I want to (again): | |||
* more reviews, especially ones from contributors | |||
* 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==== | |||
*(couldn't do much work as I was to attend JS workshop in the college and also some small subject related tests in the class) | |||
*{{Bug|961572}} - [Messages][Refactoring] Implement a new independent cancel button for information view's header | |||
** Checkin-needed done !! \o/ , Thanks Steve . | |||
*{{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 | |||
** Gecko needs to move more elements when we use translate(-100%) to move the particular panel.So, it would be better if we move complete wrapper to left/right according to panel needed to be display. | |||
** WIP. | |||
=== Day 5: 6th February === | === Day 5: 6th February === | ||
=== Day 5: 9th February === | === Day 5: 9th February === | ||
== Demos == | == Demos == | ||
== Retrospective == | == Retrospective == | ||