542272 - Email add-on to a friend feature

Status: RESOLVED FIXED

Description

[Nick Nguyen [:osunick]]

Spec here: (needs design for hi-fi mocks)

http://docs.google.com/Doc?docid=0Ad7mAOXgEBZyZGRzNnZ3YjRfMjVmbWRyZnpyNA&hl=en

Comment 1

[Neil Lee [:neilio]]

Attachment: Email this flyout modal

Here's the design for the flyout modal window - the interaction here is identical to the "share this" widget. Hitting cancel closes the modal.

After the user hits "send", a success message overlays the email this form, and then the modal disappears after 2-3 seconds - see email-this-success.png

Comment 2

[Neil Lee [:neilio]]

Attachment: Email this success message design

Comment 3

[Neil Lee [:neilio]]

Attachment: HTML version of the email recipients receive

Here's what the email recipient receives if they have HTML email enabled. If there is a custom message, the centre area that shows the add-on description is replaced by that.

Comment 4

[Nick Nguyen [:osunick]]

Hi Neil,

Sorry to drop this on you but we've decided to combine the flyout modal into the share this widget- so "email this add-on" appears as an additional option.  Adding complexity to this is that logged out users should see the option but get a note "Please log in to email this add-on" instead of the full UI of the widget.  Can we get another rev?

Also, upon further reflection, we're dumping HTML email from the spec.

Comment 5

[Neil Lee [:neilio]]

Attachment: Rejigged share this widget

Comment 6

[Neil Lee [:neilio]]

Attachment: Share this log in panel

Comment 7

[Neil Lee [:neilio]]

Attachment: Error messaging

Comment 8

[Neil Lee [:neilio]]

Attachment: Email this add-on flow

Here's the flow for the share this widget, which points out the interaction flow and various states.

Comment 9

[Neil Lee [:neilio]]

Attachment: Slightly tweaked email this panel

Rejigged email this panel with resized text field to accommodate error messaging.

Comment 10

[Nick Nguyen [:osunick]]

Perfect.  Thank you.

Comment 11

[Nick Nguyen [:osunick]]

reassigning to clouserw for assignment.

Comment 12

[Fred Wenzel [:wenzel]]

Attachment: Patch, rev. 1

Not sure why we needed to introduce sub-features like ajax login for this, but here you go.

It's huge, but hopefully easy to grasp. Each of the two emails exists as HTML and non-HTML variant.

Comment 13

[Wil Clouser [:clouserw]]

Comment on attachment 426256 [details] [diff] [review]
Patch, rev. 1

You're right, the mockups do introduce the ajax log-in which is not in the spec and which shouldn't be there.  Additionally, we're not doing the HTML emails due to image blocking in every major email client.  Sorry for not catching these sooner.  For everyone:  Mockups should not deviate from the spec without cited reasoning so please say something if you notice it.  In this case it slipped through.

Due to the number of changes in this patch please upload a new patch without HTML email and without the ajax log-in.  If a user is logged out and clicks on the email link it can just be a link that says "Log in to email this add-on to a friend"

Thanks

Comment 14

[Neil Lee [:neilio]]

I'm not exactly sure how we'd handle this without the ajax login, if you want everything within the widget itself. If we want to jump to a completely separate page, it'd work, but not integrated into the widget itself.

Comment 15

[Neil Lee [:neilio]]

Nick, can you clarify how you want this to work? If the user is logged in, the email can happen within the widget, but I don't see how we'd bring the user back to the widget post-log in without making the process really confusing.

Comment 16

[Fred Wenzel [:wenzel]]

if we make it a login link, it'll take you to the login page, and transport you back to the addon page, where you would need to click the "share this" link again.

I'm also missing where the HTML format was dropped here or in the spec, but I can remove that. clouserw is right that it will only be shown correctly on request, and gmail won't show backgrounds etc at all.

Comment 17

[Fred Wenzel [:wenzel]]

Over to Nick. Please throw it back when the specs are clear.

Comment 18

[Fred Wenzel [:wenzel]]

All right, talked to clouserw. Will remove HTML mail and AJAX login, then submit a new patch.

Comment 19

[Fred Wenzel [:wenzel]]

Attachment: Patch, rev. 2

This is the patch with half the fun stuff removed.

Comment 20

[Wil Clouser [:clouserw]]

Comment on attachment 426328 [details] [diff] [review]
Patch, rev. 2

This is awesome, thanks.

Comment 21

[Fred Wenzel [:wenzel]]

Just in case we want to look at any of this again in the future, I made the big patch an intermediary revision: r62072, followed by the fix: r62073.

Comment 22

[Nick Nguyen [:osunick]]

Hooray, no HTML email and AJAX login is correct.  Once [z] is ready we'll put AJAX login back in, with a new bug.