842487 - [MMS][User Interface] Download option settings page layout

Status: RESOLVED DUPLICATE of bug 840076

(Firefox OS Graveyard :: Gaia::SMS, defect, P1)

Description

[Steve Chung [:steveck]]

A panel or dialog for configure how terminal downloads MMS content. The options should include "automatic" (no prompting), "manual" (with prompting), and "never".

Comment 1

[Joe Cheng [:jcheng] (please needinfo)]

this is required for MMS user stories in v1.1. leo+

Comment 2

[Steve Chung [:steveck]]

Attachment: Patch for message app settings layout implementation

Comment 3

[Borja Salguero [:borjasalguero]]

Hi Steve! I would need Visual & Wireframe for checking the functionality. Thanks!

Comment 4

[Steve Chung [:steveck]]

https://www.dropbox.com/s/zc3hhd1mxi16p4w/MMS.pdf
The settings page is in p.15, thanks.

Comment 5

[Borja Salguero [:borjasalguero]]

Thanks Steve! I was checking that Wireframe and there are a lot of inconsistencies in the new header (check p. 9,10,15...). Casey, could you clarify this & update the Wireframe with the final spec.? 
On the other hand I would need the visual as well, because the 'sub-menu' of 'Edit, Settings' is implemented as an 'option screen' but I dont know if this is the final design. As well I would need the transitions to implement here in order to check them. 
Casey, Arun, I need feedback from UX in order to review this patch, Could you help us with this? Thanks!

Comment 6

[Steve Chung [:steveck]]

Hi Borja, thanks for pointing out the inconsistencies in wireframe. Except this issue, the reason why I use option screen(action menu in building-block) is to sync the behavior in email app, and the settings page animation is based on overall transition description PDF here: https://www.dropbox.com/sh/ekzrrz7um4uj6y0/NVzqB1D4hx/03%20Transitions

Comment 7

[Arun Balachandran Ganesan [:abc]]

Borja:

Casey is working on this. I will tag him as needs?info.

Thanks!
Arun

Comment 8

[Steve Chung [:steveck]]

I've modified layout based on the latest wireframe : https://www.dropbox.com/s/zc3hhd1mxi16p4w/MMS.pdf. Borja, could you pleasse review the patch again?

Comment 9

[Casey Yee]

Great :)  As mentioned in the call, the iconography should be consistent with what we already have in contacts.

Comment 10

[Borja Salguero [:borjasalguero]]

This bug is tied to the new 'generic' action screen that has been reviewed by Rick. Once we have this 'generic' action menu must be applied to the current code.

Comment 11

[Steve Chung [:steveck]]

Hi Borja, I've left some comment in github: https://github.com/mozilla-b2g/gaia/pull/8359#issuecomment-14954060 .Please take a look, thanks!

Comment 12

[Joe Cheng [:jcheng] (please needinfo)]

borja, can this be reviewed? thanks

Comment 13

[Joe Cheng [:jcheng] (please needinfo)]

leo+ as this is a part of MMS. No_UPLIFT for now before the whole MMS is completed

Comment 14

[Joe Cheng [:jcheng] (please needinfo)]

seem like there is updated spec for download options. ni? UX

Comment 15

[Borja Salguero [:borjasalguero]]

Comment on attachment 718906 [details]
Patch for message app settings layout implementation

Im gonna remove this 'r?' until having all code & UX ready in M3/M4.

Comment 16

[Dan Heberden]

Per the UX/MMS meeting/walkthrough, these options are going to be moved into the settings app. Steve is already aware of this so that the patch can be recreated/updated.