Closed
Bug 617648
Opened 15 years ago
Closed 15 years ago
Mainstream Newsletter signups - change default Country to United States, and cursor
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
FIXED
1.0
People
(Reporter: emurto, Assigned: abuchanan)
References
Details
Attachments
(1 file, 1 obsolete file)
|
2.80 KB,
patch
|
rik
:
review+
|
Details | Diff | Splinter Review |
Two changes to make for the signup centers for the Mainstream Newsletter:
1) Please change the default country to "United States" for all the EN-US signup centers for the Mainstream Newsletter:
a) Landing Page
b) First Run page
c) What's New Page
d) Mozilla.org Home Page
2) Please have the cursor immediately appear in the email address box (where they type in their email address) when they get to the signup page/box.
thank you! -Eve
| Assignee | ||
Comment 1•15 years ago
|
||
I'd just like to make clear that en-US does not necessarily mean United States. I'm sure there are plenty of international users seeing the en-US pages, so I think this will make your data less accurate.
Yes, thanks for the input. We've discussed those tradeoffs internally and wish to move forward with placing United States as the default. From analyzing the numbers, the vast majority of the signups are United States. Thanks for your help to move this forward swiftly. -Eve
Comment 3•15 years ago
|
||
To make a clarifying comment here. We are suggesting that in the drop down the default country is United States -- however, someone make select a different country from that drop down. Isnt that correct Eve? thanks.
Yes, exactly. The default shows the United states, (rather than an instructional statement about selecting their country), but they are free to scroll to any other country of their choice. -Eve
| Assignee | ||
Updated•15 years ago
|
Assignee: nobody → abuchanan
| Assignee | ||
Comment 5•15 years ago
|
||
Attachment #498833 -
Flags: review?(anthony)
| Assignee | ||
Comment 6•15 years ago
|
||
I filed a separate bug for mozilla.org changes, bug 620511
| Assignee | ||
Updated•15 years ago
|
Component: Email → www.mozilla.com
Product: Marketing → Websites
QA Contact: email → www-mozilla-com
Updated•15 years ago
|
Attachment #498833 -
Attachment is patch: true
Attachment #498833 -
Attachment mime type: application/octet-stream → text/plain
Comment 7•15 years ago
|
||
Comment on attachment 498833 [details] [diff] [review]
patch v1
I don't understand why you create autofocus.js. Can't you include that in newsletter/index.html directly? It's the only place that needs it (other than the form that already has its own JS).
Attachment #498833 -
Flags: review?(anthony) → review-
| Assignee | ||
Comment 8•15 years ago
|
||
(In reply to comment #7)
> Comment on attachment 498833 [details] [diff] [review]
> patch v1
>
> I don't understand why you create autofocus.js. Can't you include that in
> newsletter/index.html directly? It's the only place that needs it (other than
> the form that already has its own JS).
I made it separate because the code is generic and it's reusable that way. Also, I try to avoid inline JS.
Updated•15 years ago
|
Target Milestone: --- → 1.0
| Assignee | ||
Comment 9•15 years ago
|
||
Attachment #498833 -
Attachment is obsolete: true
Attachment #499360 -
Flags: review?(anthony)
Updated•15 years ago
|
Attachment #499360 -
Attachment is patch: true
Attachment #499360 -
Attachment mime type: application/octet-stream → text/plain
Comment 10•15 years ago
|
||
Comment on attachment 499360 [details] [diff] [review]
patch v2, include autofocus.js only on the needed page
Good.
If you want, you can drop 'type="text/javascript"' since it is not needed with HTML5.
Attachment #499360 -
Flags: review?(anthony) → review+
| Assignee | ||
Comment 11•15 years ago
|
||
Thanks!
Committed to trunk,
Sending en-US/newsletter/index.html
Sending en-US/newsletter-form.html
Adding js/autofocus.js
Sending js/newsletter-form.js
Transmitting file data ....
Committed revision 79841.
| Assignee | ||
Comment 12•15 years ago
|
||
(In reply to comment #10)
> Comment on attachment 499360 [details] [diff] [review]
> patch v2, include autofocus.js only on the needed page
>
> Good.
>
> If you want, you can drop 'type="text/javascript"' since it is not needed with
> HTML5.
Ah, cool. Also removed those on this page.
qa-verified-trunk:
http://www-trunk.stage.mozilla.com/en-US/newsletter/
http://www-trunk.stage.mozilla.com/en-US/firefox/connect/
https://www-trunk.stage.mozilla.com/en-US/firefox/3.6/firstrun/
https://www-trunk.stage.mozilla.com/en-US/firefox/3.6/whatsnew/ (just links to https://www-trunk.stage.mozilla.com/en-US/newsletter/)
| Assignee | ||
Comment 14•15 years ago
|
||
r79989 on production
Verified FIXED on prod:
http://www.mozilla.com/en-US/newsletter/
http://www.mozilla.com/en-US/firefox/connect/
https://www.mozilla.com/en-US/firefox/3.6/firstrun/
Status: RESOLVED → VERIFIED
Updated•13 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
Updated•13 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•