Closed Bug 1083288 Opened 8 years ago Closed 8 years ago
Lag when typing on keyboard after press Next button during configuration of Email App
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:32.0) Gecko/20100101 Firefox/32.0 Build ID: 20140923175406 Steps to reproduce: 1. Open Email App 2. On the 'New Account' screen provide valid name and email address. 3. Tap on 'Next' button. 4. After a while 'Manual setup appears. Actual results: When typing on any field (i.e. Hostname) noticable lag occurs, if user is typing continuously then typed word appears even 1-2 seconds later. Expected results: It should work fluently. If user taps on 'Manual Setup' instead of 'Next' (point 3.) then same scenario works properly - no lags. Found on: Alcatel One Touch Fire production (got from T-mobile Poland) B2G version: 18.104.22.168-prerelease master Platform version: 36.0a1 Build Identifier: 20141015040201 Git commit info: 2014-10-14 16:11:51 5f1f0960
Component: General → Gaia::E-Mail
OS: All → Gonk (Firefox OS)
Hardware: All → ARM
Initial tests on hamachi flashed with master: 1) Tapping Next from setup_account_info, has the slower typing 2) Tapping Manual Config from setup_account_info, faster typing. Perhaps there is some autoconfig stuff in play (extra network connections still going?), or something more expensive setup_progress is doing (animated progress icon?), to contribute to the slowdown. More investigation to follow.
It was the spinner in the setup_progress card. By using display: none when not in view, it restored typing speed. Asking :mcav for review, should be an easy one, just a CSS change, with notes in the pull request. This issue would also show up in 2.1 as well as master, since the change to use a setup_progress card at this point in the flow is from the xoauth changes done for bug 1059100. When I tried on the flame device, this issue is not detectable, so really an issue with the less powerful devices, like the hamachi.
Assignee: nobody → jrburke
Status: NEW → ASSIGNED
Attachment #8505796 - Flags: review?(m)
Comment on attachment 8505796 [details] [review] GitHub pull request Since I don't have a lower-end device in a state suitable for testing, I'll take your word for it that it helps. Seems sane to me.
Attachment #8505796 - Flags: review?(m) → review+
Gaia was closed when I was going to merge for the first time, then forgot this was not merged. Rebased to master, and the try thing does not complete, but it is completely unrelated to this very targeted email-only CSS change. So merged. Merged in master: https://github.com/mozilla-b2g/gaia/commit/7fe52b81bdffdb5f1371aeb1a93934b9c1063379 from pull request: https://github.com/mozilla-b2g/gaia/pull/25197
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.