diff options
author | Matthew Somerville <matthew@mysociety.org> | 2015-07-31 11:29:11 +0100 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2015-07-31 11:29:11 +0100 |
commit | 939c6d6de069654fcbcb82722de5d71fd8cb1622 (patch) | |
tree | 20ddeca55ef6706643e4c0a385f90bea06698ee4 | |
parent | 0032b9c0dccc20593dbf66dee5512f4edf7f945d (diff) |
Revert "[fixmystreet.com] Homepage button step A/B test."
This reverts commit 838708d3127bf5bd39375b19ad13b220637c851b.
Conflicts:
web/cobrands/fixmystreet/base.scss
web/cobrands/fixmystreet/layout.scss
-rw-r--r-- | templates/web/fixmystreet.com/around/postcode_form.html | 48 | ||||
-rw-r--r-- | templates/web/fixmystreet.com/header_extra.html | 20 | ||||
-rw-r--r-- | web/cobrands/fixmystreet/base.scss | 39 | ||||
-rw-r--r-- | web/cobrands/fixmystreet/layout.scss | 20 |
4 files changed, 1 insertions, 126 deletions
diff --git a/templates/web/fixmystreet.com/around/postcode_form.html b/templates/web/fixmystreet.com/around/postcode_form.html deleted file mode 100644 index afb3b66fb..000000000 --- a/templates/web/fixmystreet.com/around/postcode_form.html +++ /dev/null @@ -1,48 +0,0 @@ -[%# Identical to parent, but with an extra div and javascript for our homepage A/B test %] - -<div id="front-main"> - <div id="front-main-container"> - [% INCLUDE 'around/intro.html' %] - - <div id="homepage-ab-test"> - <p><a id="cta-report">Report a problem</a></p> - <p><a id="cta-view">Or view problems in an area you know</a></p> - </div> - - [% - question = c.cobrand.enter_postcode_text || loc('Enter a nearby street name and area'); - %] - <form action="[% c.uri_for('/around') %]" method="get" name="postcodeForm" id="postcodeForm"> - <label for="pc">[% question %]:</label> - <div> - <input type="text" name="pc" value="[% pc | html %]" id="pc" size="10" maxlength="200" placeholder="[% tprintf(loc('e.g. ā%sā or ā%sā'), c.cobrand.example_places) %]"> - <input type="submit" value="[% loc('Go') %]" id="sub"> - </div> - [% IF partial_token %] - <input type="hidden" name="partial" value="[% partial_token.token %]"> - [% END %] - </form> - </div> -</div> - -<script type="text/javascript"> -$(function(){ - $('#homepage-ab-test a').on('click', function(e){ - e.preventDefault(); - $('#homepage-ab-test').fadeOut(250, function(){ - - /* fadeIn() normal form, and focus input */ - $('#postcodeForm').fadeIn(250, function(){ - $('#pc').focus(); - }); - - /* Can't just fadeIn() because #geolocate_link - should have { display: inline-block } */ - $('#geolocate_link').css({ - opacity: 0, - display: 'inline-block' - }).animate({ opacity: 1 }, 250); - }); - }); -}); -</script> diff --git a/templates/web/fixmystreet.com/header_extra.html b/templates/web/fixmystreet.com/header_extra.html index 71c3bace5..1cce668a5 100644 --- a/templates/web/fixmystreet.com/header_extra.html +++ b/templates/web/fixmystreet.com/header_extra.html @@ -2,24 +2,4 @@ <script src="[% start %][% version('/js/jquery.cookie.min.js') %]" type="text/javascript" charset="utf-8"></script> -[%# We are conducting an A/B experiment on the homepage %] -[% IF c.req.uri.path == '/' %] -<script src="//www.google-analytics.com/cx/api.js?experiment=cX9Tbz_VRl-cZDM5WVRgTQ"></script> -<script> -var variation = cxApi.chooseVariation(), - docElement = document.documentElement, - className = docElement.className; -docElement.className = className + ' ' + 'variant' + variation; -</script> -<style> -html.variant1 #homepage-ab-test { - display: block; -} -html.variant1 #postcodeForm, -html.variant1 #front-main a#geolocate_link { - display: none; -} -</style> -[% END %] - [% INCLUDE 'tracking_code.html' %] diff --git a/web/cobrands/fixmystreet/base.scss b/web/cobrands/fixmystreet/base.scss index 6e6cbd7b9..485f0ffc2 100644 --- a/web/cobrands/fixmystreet/base.scss +++ b/web/cobrands/fixmystreet/base.scss @@ -85,45 +85,6 @@ } } -#homepage-ab-test { - display: none; // header_extra.html will override this if Analytics tells it to - - a { - display: inline-block; - cursor: pointer; - } - - #cta-report { - padding: 0.4em 1em; - margin-top: 0.5em; - font-size: 1.5em; - font-weight: bold; - color: #333; - background-color: #ffd000; - border-radius: 0.3em; - box-shadow: 0 3px 5px 0 rgba(0,0,0,0.2); - background-image: linear-gradient(rgba(255,255,255,0.3), transparent); - text-shadow: 0 1px 0 rgba(255,255,255,0.6); - border: 2px solid #BB9900; - text-decoration: none; - - &:hover, - &:focus { - background-color: #FFDB3E; - } - - &:active { - position: relative; - top: 1px; - box-shadow: 0 2px 2px 0 rgba(0,0,0,0.3); - } - } - - #cta-view { - margin: 0.5em 0 1em 0; - } -} - .unresponsive-council-warning { margin: 1em -1em; padding: 2em 2em; diff --git a/web/cobrands/fixmystreet/layout.scss b/web/cobrands/fixmystreet/layout.scss index 1d56e675c..c6f7c7df4 100644 --- a/web/cobrands/fixmystreet/layout.scss +++ b/web/cobrands/fixmystreet/layout.scss @@ -284,24 +284,6 @@ body.alertindex { } } -#homepage-ab-test { - #cta-report { - font-size: 2em; - } - - #cta-view { - line-height: 1.2em; - font-size: 1.2em; - color: #333; - border-bottom: 1px solid #917F00; - text-decoration: none; - - &:hover, &:focus { - border-bottom-color: #BD942A; - } - } -} - body.unresponsive-council { .container .content { padding: 0; @@ -351,4 +333,4 @@ body.unresponsive-council { background-image: url(images/unresponsive-council-cta-arrow@2.png); background-size: 60px; } -}
\ No newline at end of file +} |