Dan Heberden Reporter
	Description • 11 years ago

User Story:
After attaching an image #840044, a video #868225, or audio file #868190 I can tap that attachment (image/video thumbnail or audio icon) to be presented with a list of options to view/listen/play that attachment, replace that attachment, or remove that attachment.

	ayman maat :maat
	Comment 1 • 11 years ago

Dan from a UX perspective 'replace that attachment' is the lowest priority of these functions. If time is tight it can be deferred.

	greg Assignee
	Comment 2 • 11 years ago

I am assuming this bug includes making the buttons function, rather than just making the menu appear. In that case, https://bugzilla.mozilla.org/show_bug.cgi?id=840047 would be a duplicate of this one. I already have the work done for the menu to appear with non-functioning buttons, so I'm going ahead and making them work now.

	greg Assignee
	Comment 3 • 11 years ago

Having trouble finding a mock / wireframe for the Preview, anyone able to point me in the right direction?

The replace feature is blocked by https://bugzilla.mozilla.org/show_bug.cgi?id=840044

	Corey Frang [:gnarf]
	Comment 4 • 11 years ago

(In reply to greg from comment #3)
> Having trouble finding a mock / wireframe for the Preview, anyone able to
> point me in the right direction?
> 
> The replace feature is blocked by
> https://bugzilla.mozilla.org/show_bug.cgi?id=840044

Pretty sure it's just firing the open MozActivity

	greg Assignee
	Comment 7 • 11 years ago

Couldn't get this landed yesterday, I'm in office doing final code review changes and coordinating with :gnarf

	greg Assignee
	Comment 8 • 11 years ago

:gnarf37 reviewed and let me know this was good to go, should be landed soon. is anything needed from me?

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

Comment on attachment 750689 [details]
pull request on github

r=me for hash 20e4c1be5ae09b624f0c6a0afb2f3c41eb705203

go for it, thanks for the hard work. Any more tests before landing is fine for me too.

	Corey Frang [:gnarf]
	Comment 10 • 11 years ago

master: f41344254e0ee3101480890a9f26deae083bfee7

	John Ford [:jhford] CET/CEST Berlin Time
	Comment 11 • 11 years ago

v1-train: 66401e6

	Darren
	Comment 12 • 11 years ago

https://moztrap.mozilla.org/manage/cases/?filter-id=8435
https://moztrap.mozilla.org/manage/cases/?filter-id=8436
https://moztrap.mozilla.org/manage/cases/?filter-id=8437

Created test cases for issue, didn't see a UCID for the User story so marked the cases as "Messages-000".