jenkins-bot has submitted this change and it was merged.
Change subject: Golden toast wins
......................................................................
Golden toast wins
the A/B test was 50/50 so the golden toast doesn't effect editing
in a positive or negative way. It would be interesting to keep and
explore in future with other landmarks so it stays.
Change-Id: If5b66a919f2fe4d00c666c9d0a3edbc5437d3a2b
---
M javascripts/modules/editor/EditorOverlay.js
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
Robmoen: Looks good to me, but someone else must approve
JGonera: Looks good to me, approved
jenkins-bot: Verified
diff --git a/javascripts/modules/editor/EditorOverlay.js
b/javascripts/modules/editor/EditorOverlay.js
index 902852a..a890415 100644
--- a/javascripts/modules/editor/EditorOverlay.js
+++ b/javascripts/modules/editor/EditorOverlay.js
@@ -283,8 +283,7 @@
new Page( { title: title, el: $(
'#content_wrapper' ) } ).on( 'ready', M.reloadPage );
M.router.navigate( '' );
self.hide();
- // FIXME: What was the outcome of this
A-B test?
- if ( M.isTestA && self.isNewEditor ) {
+ if ( self.isNewEditor ) {
msg =
'mobile-frontend-editor-success-landmark-1';
} else {
className = 'toast';
--
To view, visit https://gerrit.wikimedia.org/r/92676
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If5b66a919f2fe4d00c666c9d0a3edbc5437d3a2b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
Gerrit-Reviewer: JGonera <[email protected]>
Gerrit-Reviewer: Robmoen <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits