Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Commit 275d70b

Browse files
committed
Bug 1691126 - Save to Pocket button test. r=gvn, a=pascalc
Differential Revision: https://phabricator.services.mozilla.com/D104262
1 parent 981c61f commit 275d70b

File tree

4 files changed

+141
-32
lines changed

4 files changed

+141
-32
lines changed

browser/components/pocket/content/main.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,11 +143,17 @@ var pktUI = (function() {
143143
var startheight = 490;
144144
var inOverflowMenu = isInOverflowMenu();
145145
var controlvariant = pktApi.getSignupPanelTabTestVariant() == "control";
146+
var loggedOutVariant = Services.prefs.getCharPref(
147+
"extensions.pocket.loggedOutVariant"
148+
);
146149

147150
if (inOverflowMenu) {
148151
startheight = overflowMenuHeight;
149152
} else {
150153
startheight = 460;
154+
if (loggedOutVariant === "button_variant") {
155+
startheight = 406;
156+
}
151157
}
152158
if (!controlvariant) {
153159
startheight = 427;
@@ -163,7 +169,7 @@ var pktUI = (function() {
163169
"about:pocket-signup?pockethost=" +
164170
Services.prefs.getCharPref("extensions.pocket.site") +
165171
"&loggedOutVariant=" +
166-
Services.prefs.getCharPref("extensions.pocket.loggedOutVariant") +
172+
loggedOutVariant +
167173
"&variant=" +
168174
variant +
169175
"&controlvariant=" +

browser/components/pocket/content/panels/js/signup.js

Lines changed: 23 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,8 @@ PKT_SIGNUP_OVERLAY.prototype = {
134134
this.dictJSON.variant = this.variant ? this.variant : "undefined";
135135
this.dictJSON.pockethost = this.pockethost;
136136
this.dictJSON.showlearnmore = true;
137+
this.dictJSON.utmCampaign = "logged_out_save_test";
138+
this.dictJSON.utmSource = "control";
137139

138140
// extra modifier class for collapsed state
139141
if (this.inoverflowmenu) {
@@ -155,18 +157,34 @@ PKT_SIGNUP_OVERLAY.prototype = {
155157
variant_a: "variant_a",
156158
variant_b: "variant_b",
157159
variant_c: "variant_c",
160+
button_variant: "signupstoryboard_shell",
161+
button_control: "signupstoryboard_shell",
158162
};
159163

160-
if (this.loggedOutVariant !== `control`) {
164+
let loggedOutVariantTemplate = variants[this.loggedOutVariant];
165+
if (
166+
this.loggedOutVariant === "button_variant" ||
167+
this.loggedOutVariant === "button_control"
168+
) {
169+
this.dictJSON.buttonVariant = true;
170+
this.dictJSON.utmCampaign = "logged_out_button_test";
171+
this.dictJSON.utmSource = "button_control";
172+
if (this.loggedOutVariant === "button_variant") {
173+
this.dictJSON.oneButton = true;
174+
this.dictJSON.utmSource = "button_variant";
175+
}
176+
}
177+
178+
if (loggedOutVariantTemplate !== `signupstoryboard_shell`) {
161179
$("body").addClass(`
162-
los_variant los_${variants[this.loggedOutVariant]}
180+
los_variant los_${loggedOutVariantTemplate}
163181
`);
164182
}
165183

166184
$("body").append(
167-
Handlebars.templates[
168-
variants[this.loggedOutVariant] || variants.control
169-
](this.dictJSON)
185+
Handlebars.templates[loggedOutVariantTemplate || variants.control](
186+
this.dictJSON
187+
)
170188
);
171189
}
172190

0 commit comments

Comments
 (0)