Closed Bug 838006 Opened 11 years ago Closed 11 years ago

[E-Mail][User Story] E-Mail image attachment

Categories

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

ARM
Gonk (Firefox OS)
defect

Tracking

(blocking-b2g:leo+, b2g18 verified)

VERIFIED FIXED
blocking-b2g leo+
Tracking Status
b2g18 --- verified

People

(Reporter: pdol, Assigned: dkuo)

References

Details

(Keywords: feature, relnote, Whiteboard: [LOE:L][by 3/15],relnote-b2g:1.1+)

Attachments

(2 files)

UCID: Email-138

User Story:
As a user, I want to be able to attach one or more image files (based on currently supported MIME types) to new emails or within replies to emails so that I can share content with my recipients.
Keywords: feature
Summary: [B2G][E-Mail][User Story] E-Mail image attachment → [E-Mail][User Story] E-Mail image attachment
Proposed flows for attachments from Gallery, Video and Music. Also contains some recommended features that are not currently requirements, including creating attachments from Contacts and Camera.
Whiteboard: u=user c=email s=v1.1-sprint-1
Whiteboard: u=user c=email s=v1.1-sprint-1 → u=user c=email s=v1.1-sprint-1 p=2
Whiteboard: u=user c=email s=v1.1-sprint-1 p=2 → u=rmacdonald@mozilla.com c=email s=v1.1-sprint-1 p=2
Blocks: 840542
as dkuo took bug Bug 836497, assign this one to dkuo. 
if anyone has already started working on it, you may discuss with dkuo to take this
Assignee: nobody → dkuo
Whiteboard: u=rmacdonald@mozilla.com c=email s=v1.1-sprint-1 p=2 → u=rmacdonald@mozilla.com c=email s=v1.1-sprint-1 p=2 [LOE:L]
Whiteboard: u=rmacdonald@mozilla.com c=email s=v1.1-sprint-1 p=2 [LOE:L] → [LOE:L]
This flows have been updated and placed on drop box here:

https://www.dropbox.com/s/5giljes7ao9gyrg/email-attachments.pdf

Two items to flag in this version...

First, the selection tools for video, contacts and music will likely only allow users to pick one file at a time. For long lists, we felt that the UX for multiple selections might become overly complex. As a result, if a user wants to attach multiple files, they would need to attach a second file from the message view add attachment "+" button. The screens will be updated this week to reflect this.

Second, the spec contains a proposal for a document manager to handle unrecognized attachment types. This would allow users to save, forward and delete attachments that are not compatible with the core set of apps. This is not in scope and is for discussion purposes only.
Whiteboard: [LOE:L] → [LOE:L][by 3/15]
Andrew and David,

This patch modify both email and gallery app, so I think it will be better for this patch to be reviewed by both of you, thanks.
Attachment #721361 - Flags: review?(dflanagan)
Attachment #721361 - Flags: review?(bugmail)
Comment on attachment 721361 [details]
Modify gallery and email to support attaching image files from picker

It seems that there are too many review requests for David, so I will cancel this one first for David to ease his task loading.
Attachment #721361 - Flags: review?(dflanagan)
Comment on attachment 721361 [details]
Modify gallery and email to support attaching image files from picker

Thanks for the patch!  This is looking pretty good, but we need to address the localization issue and providing default filenames for camera/wallpaper before I can sign off.  Please request review again when that's fixed up.
Attachment #721361 - Flags: review?(bugmail)
Comment on attachment 721361 [details]
Modify gallery and email to support attaching image files from picker

Andrew, I have update my patch base on your comments.

The localization issue is fixed by using the plural functionality of l10n.js.
(Actually I didn't know we have the plural functionality before you told me! thanks)

And the issue of the default filenames for camera/wallpaper is also fixed, for the blobs from camera/wallpaper, they do have names, we just need to pass it in the activity object, so that's how I fixed it.

It's still possible that we are unable get the name field from data.blob.name or data.name, but it won't be a problem to this user story, so I filed bug 848855 to record it and we can fix that later, after ux define it.
Attachment #721361 - Flags: review?(bugmail)
Blocks: 824749
Comment on attachment 721361 [details]
Modify gallery and email to support attaching image files from picker

r=asuth.  The styling of the aggregate file size is a little wonky, but we'll leave that up to visual design to fix.  Apologies about the delay on the follow-up review; things have been crazy and it fell through the cracks.  Please do feel free to ping me if I'm letting a review linger and I haven't explicitly given an ETA on when I'll get to the review!

I'll merge this now.
Attachment #721361 - Flags: review?(bugmail) → review+
Andrew, do we still want this uplifted?
Flags: needinfo?(bugmail)
v1.0.1: landed but disabled in Bug 851124
will land in v1.1 tomorrow at the latest.
Flags: needinfo?(bugmail)
Landed in v1-train as part of Bug 851124 big uplift.
I believe we have test case coverage for this, right?
Flags: needinfo?(nhirata.bugzilla)
Flags: in-moztrap?
Flags: needinfo?(nhirata.bugzilla)
Flags: in-moztrap?(nhirata.bugzilla)
Flags: in-moztrap?
Forgot to flip the bit: #6893

bug 855890 was found when testing.

Marking as verified.  Note that attachments received by hotmail is not recognized as images, because the extension wasn't preserved (see bug 855890)
Status: RESOLVED → VERIFIED
Flags: in-moztrap?(nhirata.bugzilla) → in-moztrap+
Gecko  http://hg.mozilla.org/releases/mozilla-b2g18/rev/f9f11b8cbf8a
Gaia   663101b6eb809383e5882d9bc3868a923a57998a
BuildID 20130401070203
Version 18.0
will uplift this properly tomorrow.
Flags: needinfo?(felash)
v1-train: 433db9c80066986e6597346f3c20f3b249f86e29

should be really fixed now, but please test properly as there were conflicts in the gallery.
Flags: needinfo?(felash)
Rob and Przemek have been working on the UX for this, in the bugs I've added that block this one. We expect completion on VxD early next week (April 22-23, 2013).

This cannot ship as currently implemented.
Depends on: 860021
Depends on: 860092
Depends on: 860008
verified fixed on Leo with:

Gecko  http://hg.mozilla.org/releases/mozilla-b2g18/rev/0c71cbc5fe0c
Gaia   a7b0810580afc734f3d5e441914fe895f9c1923e
BuildID 20130508230207
Version 18.0
Keywords: relnote
Whiteboard: [LOE:L][by 3/15] → [LOE:L][by 3/15],relnote-b2g:1.1+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: