Closed Bug 1109250 Opened 10 years ago Closed 10 years ago

add newsletter signup fields and controls to footer of marketplace

Categories

(Marketplace Graveyard :: Consumer Pages, defect, P2)

x86
macOS
defect

Tracking

(Not tracked)

VERIFIED FIXED
2014-12-16

People

(Reporter: ddurst, Assigned: mat)

References

Details

We need to add form fields and controls to the marketplace footer for newsletter subscriptions.

The UX is described in http://people.mozilla.org/~ehunt/newsletter_signup_Nov14/newsletter_v1.8.pdf.

The form fields are:
* name:newsletters; type:hidden; value:(FxOS)?'marketplace':'mozilla-and-you'
* name:source_url; type:hidden
* name:email; type:email (pre-fill if possible)
* name:country; type:select options:2-letter country codes (pre-fill if possible)
* name:lang; type:select; options:2-letter language codes (pre-fill if possible)
* name:privacy; type:checkbox

As there are no controls for these fields, their values are set by default:
* optin=Y
* format=H
* trigger_welcome=Y
* sync -- omitted (optional)
https://github.com/mozilla/fireplace/commit/640df6a5963f9bf6a0deac59fe2dd2594ef7c443
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Depends on: 1110551
Depends on: 1110525
Blocks: 1110561
No longer blocks: 1097152
Paul -- where can we refer to a canonical list of languages to be used for newsletter subscriptions?
Status: RESOLVED → REOPENED
Flags: needinfo?(pmac)
Resolution: FIXED → ---
No longer depends on: 1110525
It depends on which newsletter to which you'll be subscribing people as some newsletters are more well localized than others. The full list for each newsletter can be had from the data in the newsletters API:

https://basket.mozilla.org/news/newsletters/

This contains data about all newsletters including supported languages.
Flags: needinfo?(pmac)
"Get Firefox news! Sign up for our newsletter" is missing punctuation:

https://www.dropbox.com/s/i89f0jbmv7tr4dk/Screenshot%202014-12-12%2013.14.05.png?dl=0

––

Focussing the email field or submit button causes the entire section to be re-rendered/re-toggled (even if it's already visible):

https://www.dropbox.com/s/hz290cbe379n8rp/Screenshot%202014-12-12%2013.14.55.png?dl=0
> "Get Firefox news! Sign up for our newsletter" is missing punctuation:

Copy in the PDF didn't have a punctuation at the end either, so I didn't know what to do. Do you think there should be a period ?

> Focussing the email field or submit button causes the entire section to be re-rendered/re-toggled (even if it's already visible):

Can you expand on that ? Not sure what you mean.
(In reply to Mathieu Pillard [:mat] from comment #5)
> > "Get Firefox news! Sign up for our newsletter" is missing punctuation:
> 
> Copy in the PDF didn't have a punctuation at the end either, so I didn't
> know what to do. Do you think there should be a period ?

To be honest, I'm not sure. I would probably keep the heading as "Get Firefox news!" and have a sub-heading be "Sign up for our newsletter" (without punctuation). I'll defer to Scott or someone re: punctuation; I'm not an expert with exciting headings!

> > Focussing the email field or submit button causes the entire section to be re-rendered/re-toggled (even if it's already visible):
> 
> Can you expand on that ? Not sure what you mean.

Mark fixed this on Friday: https://github.com/mozilla/fireplace/commit/0d4d6521
Flags: needinfo?(sdevaney)
Priority: -- → P2
Well yeah, technically we should have a period at the end of the sentence, so it would read like so:

Get Firefox news! Sign up for our newsletter. 

But actually, the thing that bugs me a bit more is "E-Mail" in the field. That hyphen is pretty 1998, no? Why not just "Email"? However "E-Mail" may be in accord with the broader Mozilla style as established by the brand team?
Flags: needinfo?(sdevaney)
The photo here is a little out of date. E-Mail has already been switched to Email.
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
Verified as fixed . The message is "Get Firefox news! Sign up for our newsletter" I am not sure about the punctuation but it looks good on Marketplace : screencast.com/t/f9y2jBmJzpq
Status: RESOLVED → VERIFIED
Depends on: 1113354
You need to log in before you can comment on or make changes to this bug.