Closed
Bug 826386
Opened 11 years ago
Closed 11 years ago
[B2G][Email][Accouns] If you have 10 or more accounts set on a device you are unable to select any email account after the 9th one from a top and to scroll down the email list to see all available email accounts.
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect)
Tracking
(blocking-basecamp:-, b2g18+ fixed, b2g18-v1.0.1 fixed)
VERIFIED
FIXED
blocking-basecamp | - |
People
(Reporter: ndavidson, Assigned: steveck)
References
Details
(Whiteboard: uxbranch, landed in uxbranch, qa-verified)
Attachments
(2 files)
Unagi build 20130103070201 In email account select menu (If you have 10 or more accounts set on a device) you are unable to select email accounts after the 9th one from a top. Also unable to scroll down or up the email list to see all available email accounts. Repro steps: 1. Set up 10 or more email accounts on a device 2. Go to accounts menu so you see all 10 accounts Expected result: You will be able to scroll down the email accounts list and select email accounts after the 9th one from a top. Actual result: you see 10 accounts on page, the 10th one from a top covered by gray banner with gear (settings) icon and unaccessible.( see pic.) for easy repro here is a list of test emails I used: Nickdeetest1@gmail.com nickdeetest2@gmail.com Nickdeetest3@hotmail.com NickDeeTest4@yahoo.com Nickdeetest5@aol.com Nickdee77@yahoo.es Qsummer2000@hotmail.es Qwinter2000@gmail.com Qsummer2000@gmail.com Milatest2000@yahoo.com Password for all emails: QQ111111
Updated•11 years ago
|
Component: Gaia → Gaia::E-Mail
QA Contact: nhirata.bugzilla
Reporter | ||
Comment 1•11 years ago
|
||
can be related to Bug 805590 - Accounts missing from E-Mail Accounts pane
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → schung
blocking-basecamp: --- → ?
Comment 2•11 years ago
|
||
Triage: BB-, tracking-b2g18+. most users won't have 10 accounts
blocking-basecamp: ? → -
tracking-b2g18:
--- → +
Reporter | ||
Comment 3•11 years ago
|
||
True most users won't have 10 accounts but those who do will complain... :-) I see scrolling been enabled, but there is still the issue user is unable to select last account on the bottom of the list because it is covered with gray banner with Settings(gear)icon. I suggest to make email list shorter so it is only shows 9 emails in list or make it able to scroll above the banner on a bottom.
Reporter | ||
Comment 4•11 years ago
|
||
this is design issue Settings banner on a bottom covering last email in a list (banner takes 6 squares according to developer grid - see pic) and each email box takes 5 squares. I suggest to move scrolling-up stop point from very bottom of the page to 5 or 6 squares from a bottom. See PIC- attachment.
Updated•11 years ago
|
OS: Windows 7 → Gonk (Firefox OS)
Hardware: x86_64 → ARM
Comment 5•11 years ago
|
||
Landed in uxbranch: https://github.com/gordonbrander/gaia/commit/b671c5addbba4668681e01d8c8259056a4b71109
Whiteboard: uxbranch, landed in uxbranch
qa-verified... tested in pt-BR with 12 accounts: checked scrolling, switching accounts
Whiteboard: uxbranch, landed in uxbranch → uxbranch, landed in uxbranch, qa-verified
I suppose I could have tried to see what the max is... but I'm not sure what the point would be...
Comment 8•11 years ago
|
||
There is no max; you could be in for a long day of typing if you try that :)
Reporter | ||
Comment 9•11 years ago
|
||
The issue is when you have more then 10 emails you are unable to select the last one you added because it will appear on a bottom on a page covered with "settings banner" when you scrolling email list up - stop point of scrolling located on a bottom of the page,(covered with "settings banner") but it should be in line with upper edge of "settings banner" so you can select the last email on a list. see last attachment for visual.
Assignee | ||
Comment 10•11 years ago
|
||
The patch already landed in UX-branch, but I'm not sure if it will be cherry-picked to master...Need more information for UX-branch owner?
Flags: needinfo?(sjochimek)
Comment 11•11 years ago
|
||
This bug is waiting to be merged in master: https://github.com/mozilla-b2g/gaia/pull/7818 I'll talk to Vivien to merge it asap.
Flags: needinfo?(sjochimek)
Comment 12•11 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/eb803368dfd558fc03aac5fb89ca3376d1f90497
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 13•11 years ago
|
||
Issue still repro on Build ID: 20130125070201 Kernel: Dec 5 Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/94a2d6fcdfde Gaia: 6369dbf33b622faf4b4d176fed30b77c5c319dfc
Reporter | ||
Comment 14•11 years ago
|
||
Only thing was fixed in this bug is scrolling option in email accounts list was implemented. If you have 10+ accounts on a device - issue still exists with last email that set up on a device is covered by Settings banner and unaccessible by user. Closing this bug and reopening new one to fix this issue. Bug 840306
Reporter | ||
Updated•11 years ago
|
Status: RESOLVED → VERIFIED
Comment 15•11 years ago
|
||
I was not able to uplift this bug to v1-train. If this bug has dependencies which are not marked in this bug, please comment on this bug. If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval. Otherwise, if this is just a merge conflict, you might be able to resolve it with: git checkout v1-train git cherry-pick -x -m1 eb803368dfd558fc03aac5fb89ca3376d1f90497 <RESOLVE MERGE CONFLICTS> git commit
Flags: needinfo?(schung)
Comment 16•11 years ago
|
||
The specific master commit for this was 508aa96 (the PR had commits for several other apps). The email part was actually landed as part of Bug 851124 uplift to v1.0.1 and will be to v1-train tomorrow at the latest.
status-b2g18-v1.0.1:
--- → fixed
Flags: needinfo?(schung)
Comment 17•11 years ago
|
||
Landed in v1-train as part of Bug 851124 big uplift.
status-b2g18:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•