Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: html/static/css/firstRun.css

Issue 5545110112567296: Issue #1204 - Remove A/B testing for donation button in firstrun page in IE (Closed)
Patch Set: Created Oct. 27, 2014, 1:16 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | html/static/js/firstRun.js » ('j') | html/static/js/firstRun.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: html/static/css/firstRun.css
===================================================================
--- a/html/static/css/firstRun.css
+++ b/html/static/css/firstRun.css
@@ -11,7 +11,7 @@
#wrapper,
#logo,
.share-image,
-#share-images2,
+#share-images,
#share-donate
{
display: inline-block;
@@ -61,16 +61,14 @@
list-style-image: url(../img/checkmark.png);
}
-#share1,
-#share2
+#share,
Thomas Greiner 2014/11/03 17:14:12 Nit: Remove the trailing comma.
{
display: none;
font-size: 120%;
margin-top: 40px;
}
-html.share-variant-1 #share1,
-html.share-variant-2 #share2
+html.share #share,
Thomas Greiner 2014/11/03 17:14:12 This block was only used to show the correct varia
{
display: block;
}
@@ -81,7 +79,7 @@
height: 82px;
}
-#share-images2
+#share-images
{
box-shadow: 0 0 4px 3px #EEEEEE;
border-radius: 5px;
@@ -90,7 +88,7 @@
margin-bottom: 30px;
}
-#share-images2 *
+#share-images *
{
vertical-align: middle;
}
@@ -107,11 +105,15 @@
padding: 2px 10px;
background-image: url(../img/button-background/donate.png);
background-repeat: repeat-x;
+ vertical-align: top;
+ margin: 35px 0px;
}
-#share2-connection
+#share-connection
{
- margin: 0px 20px;
+ vertical-align: top;
+ display: inline-block;
+ margin: 35px 20px;
Thomas Greiner 2014/11/03 17:14:12 Same as above: Why are these changes necessary?
}
.share-facebook
« no previous file with comments | « no previous file | html/static/js/firstRun.js » ('j') | html/static/js/firstRun.js » ('J')

Powered by Google App Engine
This is Rietveld