Closed
Bug 965778
Opened 11 years ago
Closed 11 years ago
[Messages] Display contact photo in each message shown in a conversation view
Categories
(Firefox OS Graveyard :: Gaia::SMS, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 963017
People
(Reporter: vittone, Unassigned)
Details
Attachments
(1 file)
109.94 KB,
image/png
|
Details |
In the message thread, after the visual refresh, the sender has a picture.
As Ayman commented: "We do not have the facility on the phone at the moment for users to add an image of themselves"
In order to implement the design proposal as is, we need to develop a feature related to this.
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → 1.4?
Comment 1•11 years ago
|
||
I'm not really sure this should be in the SMS app. I think it should be in the Settings app, and asked already during FTU. The image would then be either stored in Settings or DeviceStorage and the SMS app or other apps could retrieve it.
So there are more UX implication (hence needinfo the broader UX team) and technical implications (hence needinfo Vivien).
Flags: needinfo?(firefoxos-ux-bugzilla)
Flags: needinfo?(21)
Updated•11 years ago
|
Summary: [SMS] Ability to add sender's picture to show in the Messages App → [Messages] Display contact photo in each message shown in a conversation view
Comment 2•11 years ago
|
||
Flagging Jacqueline on FTU and Neo on Settings, though please note that it is Chinese New Year and so Taipei UX will likely not respond until post holiday.
Flags: needinfo?(nhsieh)
Flags: needinfo?(jsavory)
Flags: needinfo?(firefoxos-ux-bugzilla)
Comment 3•11 years ago
|
||
Please assign to another UX designer. Thanks
Flags: needinfo?(nhsieh) → needinfo?(swilkes)
Comment 4•11 years ago
|
||
Flagging Omega to comment on whether or not this is appropriate for Settings. Jacqueline can still cover the FTU comment but we've just wrapped up a work week so responses are a bit behind here.
Flags: needinfo?(swilkes) → needinfo?(ofeng)
Comment 5•11 years ago
|
||
ni? Victoria, is the sender's picture necessary? I mean, is this defined in the Messages UX spec, or coming from visual refresh?
Currently we don't have a section for personal information in Settings. If the sender's picture is necessary, we can add an section for it. Or the better idea I think is adding a contact "Me" in Contacts app.
Flags: needinfo?(ofeng) → needinfo?(vpg)
Comment 6•11 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #1)
> I'm not really sure this should be in the SMS app. I think it should be in
> the Settings app, and asked already during FTU. The image would then be
> either stored in Settings or DeviceStorage and the SMS app or other apps
> could retrieve it.
>
> So there are more UX implication (hence needinfo the broader UX team) and
> technical implications (hence needinfo Vivien).
Well, can't we simply let the user create his own contact ? Then he can assign any picture he want.
Flags: needinfo?(21)
Comment 7•11 years ago
|
||
Yep I didn't think of it, this looks fine.
But then we need a way to say "get my own contact card" from an activity or mozContact.
Comment 8•11 years ago
|
||
(In reply to Omega Feng [:Omega] from comment #5)
> ni? Victoria, is the sender's picture necessary? I mean, is this defined in
> the Messages UX spec, or coming from visual refresh?
> Currently we don't have a section for personal information in Settings. If
> the sender's picture is necessary, we can add an section for it. Or the
> better idea I think is adding a contact "Me" in Contacts app.
This comes from a proposal made for the Visual Refresh. It's not a feature of SMS but yes a need derivated from that proposal.
Thanks.
Flags: needinfo?(vpg)
Updated•11 years ago
|
Flags: needinfo?(wmathanaraj)
Comment 11•11 years ago
|
||
In the lastest spec attachment 8394791 [details], there is no need to show contact photo in the conversation view. I'll close it as invalid first. Please reopen it if spec changes and it becomes necessary for conversation view.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Comment 12•11 years ago
|
||
Hi,
Not needed for 2.0 but something to be consider for future releases, please see https://bugzilla.mozilla.org/show_bug.cgi?id=963017#c3 comment from visual design team. Bug 963017 was added to backlog so marking this bug as duplicate of that.
Resolution: INVALID → DUPLICATE
Comment 13•11 years ago
|
||
As this won't be included in FTU, but instead contacts or settings, I'm removing the flag.
Flags: needinfo?(jsavory)
You need to log in
before you can comment on or make changes to this bug.
Description
•