Bug 816566 - tabzilla fundraising t-shirt promo #547

Merged
merged 3 commits into from Dec 21, 2012

2 participants

@craigcook
Mozilla member

No description provided.

@sgarrity sgarrity and 1 other commented on an outdated diff Dec 21, 2012
apps/tabzilla/templates/tabzilla/tabzilla.js
@@ -528,10 +528,10 @@ Tabzilla.closeSubmenu = function($item, $menu)
Tabzilla.content =
'<div id="tabzilla-contents">'
+ ' <div id="tabzilla-promo">'
- + ' <div class="snippet" id="tabzilla-promo-mobile">'
- + ' <a href="https://www.mozilla.org/firefox/fx/?WT.mc_id=tzfxmobile&amp;WT.mc_ev=click#mobile">'
- + ' <h4>{{ _('Fast. Smart. Safe.')|js_escape }}</h4>'
- + ' <p>{{ _('Get Firefox for Android')|js_escape }} »</p></a>'
+ + ' <div class="snippet" id="tabzilla-promo-donate">'
+ + ' <a href="https://donate.mozilla.org/Firefox-T-Shirt-8?WT.mc_id=tzdonate&amp;WT.mc_ev=click">'
+ + ' <h4>{{ _('Support Mozilla and Get a T-shirt to Boot!')|js_escape }}</h4>'
@sgarrity
sgarrity added a line comment Dec 21, 2012

This doesn't match the sentence-case capitalization used elsewhere on the site, but that may be intentional (and probably isn't a blocker).

@craigcook
Mozilla member
craigcook added a line comment Dec 21, 2012

This was the copy as provided, but you're right, official style guide says sentence case for headings. I'll fix that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@sgarrity

r+ some more!

@craigcook craigcook merged commit 080b3cb into mozilla:master Dec 21, 2012

1 check passed

Details default Jenkins build 'bedrock_github' #98 has succeeded
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment