Steve Chung [:steveck] Assignee
	Description • 11 years ago

Follow up bug for Bug #933131. Visual assets and guide in bug 919977. Except for some layout adjustment and image assets replacement, the delivery tick in message bubble will be removed per ux request.

	Steve Chung [:steveck] Assignee
	Comment 1 • 11 years ago

Hi Jose, this view is the original group participient view with the participient list. This view comes from activating the group participient header, and report view also reuse this contact list for the basic structure. Since you specify some visual requirement for contact list in the report view, should we sync the changes in group participient view?

	José Vittone
	Comment 2 • 11 years ago

Hi Steve,

The contact list in the message report view is also aligned with Contacts app. It seems that this is a similar case. I can't access to the view you shared, could you tell me how to do it if it is possible on my phone?

Thanks,
Jose

	Julien Wajsberg [:julienw]
	Comment 3 • 11 years ago

José: bug 933131 has not landed yet, that's why you don't see the panel on builds yet.

	Steve Chung [:steveck] Assignee
	Comment 4 • 11 years ago

Hi Julien, I polished the layout and fixed the re-translate issue for some date related element. The fixing seems works right after chaging the language but still need you help to see if that is the proper place/method to do the re-translate, thanks.

	Julien Wajsberg [:julienw]
	Comment 5 • 11 years ago

Comment on attachment 8356645 [details]
Link to github

added comments on github

overall this looks good :)

	Steve Chung [:steveck] Assignee
	Comment 6 • 11 years ago

Comment on attachment 8356645 [details]
Link to github

Hi Julien, I've updated the patch based on your suggestions, thanks for the comments!

	Julien Wajsberg [:julienw]
	Comment 7 • 11 years ago

I've found 2 issues while reviewing. I think each of them belongs to a different bug:

* when you long press on a message that is in the bottom, then the menu item are triggered. This is a APZC bug, bug 959199.

* the "sent" timestamp is supposed to be displayed somewhere in the delivery panel, but I think we missed it in the original bug, and in this bug as well. Can we add it in a follow-up bug ?

	Julien Wajsberg [:julienw]
	Comment 8 • 11 years ago

Comment on attachment 8356645 [details]
Link to github

r=me, with the small nit

please also file the follow-up bug

thanks!

	Steve Chung [:steveck] Assignee
	Comment 9 • 11 years ago

Thanks for the review, follow up bug 959449 created.
Landed in master: 3a19266097b436aa2e23a2bc3c9d4383d27e88bc

	Julien Wajsberg [:julienw]
	Comment 10 • 11 years ago

Steve, see comment 7, I'd like a follow-up bug to display the "sent" timestamp too! (which was implemented in Gecko in bug 901457)

	Steve Chung [:steveck] Assignee
	Comment 11 • 11 years ago

(In reply to Julien Wajsberg [:julienw] from comment #10)
> Steve, see comment 7, I'd like a follow-up bug to display the "sent"
> timestamp too! (which was implemented in Gecko in bug 901457)

I create another bug 961574 for sent timestamp.