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

Based on the latest message app wireframe v8.0 page 25, we need to display the message sent timestamp for received message report(and with a differnt report layout). This issue is pure gaia work and need the additional message sent timetamp property landed in bug 901457.

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

To QA: this won't be in 1.3 as all the delivery report panel feature has been moved to 1.4.

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

Hi Jose,
In the wireframe spec v8.0 p.25, we need a different report layout for received message. Do you have a precise visual spec for the received message report layout? Thanks!

	José Vittone
	Comment 3 • 11 years ago

Hi Steve, actually the layout for recieved or sent message report is almost the same. From my point of view are only text changes, there's no new element or style on this variation. Attaching the message report SPEC, just in case.

Please tell me if you need something else.

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

Well, in the WF page 25, the received message view is slightly different from the spec you gave. Received message will have sent/received timestamp at the bottom of the page and no delivery status at top.

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

Hi Borja, this is a sentTimestamp feature which already landed in gecko month ago. We need this for showing the received message sent timestamp in report view(and with some layout adjustment), thanks!

	José Vittone
	Comment 6 • 11 years ago

Let make differences explicit. Here we go.

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

Comment on attachment 8374831 [details]
SMS_MMS_messageReport.png

Hi Ayman, this visual seems different from your WF spec. Do we need to put the sent/receive timestamp inside the contact information block or outside the block?

	ayman maat :maat
	Comment 9 • 11 years ago

Comment on attachment 8374831 [details]
SMS_MMS_messageReport.png

(In reply to Steve Chung [:steveck] from comment #8)
> Comment on attachment 8374831 [details]
> SMS_MMS_messageReport.png
> 
> Hi Ayman, this visual seems different from your WF spec. Do we need to put
> the sent/receive timestamp inside the contact information block or outside
> the block?

Hey Steve. 

This visual is for received messages. If you refer to frame '3. Incoming message report' on page 25 of Wireframe pack FFOS_MessageApp_V1.3_20131125_V8.0 you will see the the visual Jose has provided actually has the same information structure as defined in the Wireframes.

	Borja Salguero [:borjasalguero]
	Comment 10 • 11 years ago

Comment on attachment 8373927 [details]
Link to github

R+!

	Borja Salguero [:borjasalguero]
	Comment 11 • 11 years ago

Comment on attachment 8373927 [details]
Link to github

Retesting because I've seen something weird in the tests.

	Borja Salguero [:borjasalguero]
	Comment 12 • 11 years ago

Steve, rebasing your code with master I found the following:

  1) [sms] SMS App Unit-Test Threads-list Threads-list rendering Render unread style properly:
     Error: expected 2 to equal 1
      at chaiAssert (http://sms.gaiamobile.org:8080/common/test/helper.js:33)
      at equal (http://sms.gaiamobile.org:8080/common/vendor/chai/chai.js:1250)
      at assertNumOfElementsByClass (http://sms.gaiamobile.org:8080/test/unit/sms_test.js:105)

Could you take a look? Thanks!

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

Actually I've seen this on "normal" master too, so maybe we should fix this in another bug.

Clearly this is strange because Travis is fine :(

	Borja Salguero [:borjasalguero]
	Comment 14 • 11 years ago

Comment on attachment 8373927 [details]
Link to github

Agree with Julien. Probably we should check why tests are failing and fix this in other bug. R+.

Steve, let's merge this when the branch will be ready

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

(In reply to Borja Salguero [Back to Gaia!] [:borjasalguero] from comment #14)
> Comment on attachment 8373927 [details]
> Link to github
> 
> Agree with Julien. Probably we should check why tests are failing and fix
> this in other bug. R+.
> 
> Steve, let's merge this when the branch will be ready

Thanks, but it seems we have to wait for a long time for landing... :(

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

Landed in master: 3551052d34d6db3568091c3bc65a1a20a4625f71