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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-basecamp:-, b2g18+ fixed, b2g18-v1.0.1 fixed)

VERIFIED FIXED
blocking-basecamp -
Tracking Status
b2g18 + fixed
b2g18-v1.0.1 --- fixed

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
Component: Gaia → Gaia::E-Mail
QA Contact: nhirata.bugzilla
can be related to Bug 805590 - Accounts missing from E-Mail Accounts pane
Assignee: nobody → schung
blocking-basecamp: --- → ?
Triage: BB-, tracking-b2g18+. most users won't have 10 accounts
blocking-basecamp: ? → -
tracking-b2g18: --- → +
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.
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.
OS: Windows 7 → Gonk (Firefox OS)
Hardware: x86_64 → ARM
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...
There is no max; you could be in for a long day of typing if you try that :)
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.
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)
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)
Issue still repro on 
Build ID: 20130125070201
Kernel: Dec 5
Gecko:  http://hg.mozilla.org/releases/mozilla-b2g18/rev/94a2d6fcdfde
Gaia:   6369dbf33b622faf4b4d176fed30b77c5c319dfc
Blocks: 840306
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
Status: RESOLVED → VERIFIED
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)
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.
Flags: needinfo?(schung)
Landed in v1-train as part of Bug 851124 big uplift.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: