806599 - [unagi] [email] Sent mail folder doesn't tell me who I sent mail to

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::E-Mail, defect, P3)

Description

[Marcia Knous [:marcia]]

Unagi device using:

Gaia: ddafa86fc92914306be99c8458362cfc96d44e25
Gecko: f950df2007d07a8a6f5c5d5d29383c832249a22e

STR:
1. Set up a mail account.
2. Send mail to someone.
3. Look at the sent mail folder.

Expected: I would be able to see who I sent the email to.
Actual: I cannot tell who the recipient was

Comment 1

[Steve Chung [:steveck]]

Hi Marcia,

You can click the subject to expand the message detail, than you will know the recipient address. I know it would be a little bit inconvenient while viewing the sent message, but it is the default behavior that we only display the sender address at first. Casey, do you have any comment about this issue?

Comment 2

[Dylan Oliver [:doliver]]

blocking-

Very nice to have but not blocking for v1 -- we'll have to live with the workaround.

Comment 3

[Casey Yee]

I think it's important that I can see quickly who I sent emails to.

Comment 4

[Marcia Knous [:marcia]]

I filed this on an older build - let me check an updated build to see what the current behavior is because I think this might have changed since 10-29.

Comment 5

[Andrew Sutherland [:asuth] (he/him)]

We definitely have not changed the implementation.

Casey, can you update the wireframes for this (special) case?  The guidance I am primarily looking for is whether you only want us to show the name of the first recipient, or if we should try and cram any additional recipients in there as well, delimited by a comma or the like.

There are some data storage/scaling issues which is my main concern.  Right now we store the list of recipients with our data on the body to reduce the size of the header storage used.  We could move all of that information over to the header, or we could just duplicate some of it.

The main cases I see are:

A) 1 name, so: Alice Apple
B) 1 name plus a count of other recipients, so: Alice Apple +5 others
C) finite set of names, say capped at 3 because we assume even when run on a larger device, that should be as much as visually fits: Alice Apple, Bob Banana, Chuck Canteloupe
D) all names, although truncated at some large number of names so we don't fill the DOM up with stuff it won't possibly display.

Of course, you could also change the visual display of the messages in the sent folder so we actually have much much more space to display the list of recipients.


I think my personal inclination at this point would be just to move the list of recipients from our body storage to our header storage.  We would want to do that eventually anyways for devices with more screen real estate, and our headers are sufficiently tightly packed that I don't see a real issue with doing so.  Also, the information should compress fairly well using our block storage mechanism.

Comment 6

[Jason Smith [:jsmith]]

Please explicitly minus bugs. Do not unnom bugs. Thanks.

Comment 7

[David Scravaglieri [:scravag]]

It's not a minused bug it's just postponed after January 15th. This is the reason why we used kilimandjaro tag.

Comment 8

[Jason Smith [:jsmith]]

Thank you alex for improving my bugzilla sanity. I was about suggest using the tracking flag.

Comment 9

[Andrew Sutherland [:asuth] (he/him)]

From https://bugzilla.mozilla.org/show_bug.cgi?id=798292#c15 that I think was meant to be made here, Casey says:

Spec has been updated with details on how we should be handling the Recipient name in the sent folder.

See page. 5 of spec.
https://www.dropbox.com/s/vaf9d6ppbw58zi1/mail-main.pdf

The [name] field should show the recipients name.  If the name of recipient is not available we should show the email address.

If there is more than a single recipient in the email (multiple To, Cc or Bcc recipients) we should show [x] recipients.   The user can view the full recipient list in the email by tapping on the message to open.

Comment 10

[Casey Yee]

Thanks for relocating the comment!

Comment 11

[ayman maat :maat]

[TEF_REQ] as Feature required for TEF build.

Comment 12

[Andrew Sutherland [:asuth] (he/him)]

I've partially addressed this bug with the fix for the local drafts implementation on bug 838012.  We'll now show the first recipient in outgoing (drafts, local drafts, sent) folders.  That's not what the spec agreed upon, but the change that was landed was very very easy (thanks to James Lal's fixes for lazy bodies!).

Comment 13

[Dylan Oliver [:doliver]]

triage: not blocking, this was partially addressed in another fix and is in better shape now. we probably don't have the screen real estate to implement the UX spec.

Comment 14

[BMO Automation]

Firefox OS is not being worked on