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

Unified Diff: static/css/index.css

Issue 29322769: Issue 2844 - Create Microsoft Edge coming soon landing page (Closed)
Patch Set: Addressing more comments. Created Sept. 23, 2015, 12:36 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
Index: static/css/index.css
===================================================================
--- a/static/css/index.css
+++ b/static/css/index.css
@@ -69,7 +69,7 @@
display: inline-block;
}
-#video, #maxthon-instruction
+#video, #maxthon-instruction, #edge-teaser
{
background: #ffffff;
padding: 10px;
@@ -80,7 +80,7 @@
#maxthon-instruction,
html[lang^="zh_"] #content.maxthon #video,
#content.maxthon #feature-free,
-#feature-maxthon
+#feature-maxthon, #edge-subscription, #edge-teaser
{
display: none;
}
@@ -96,6 +96,16 @@
display: inline-block;
}
+#content.edge #video
+{
+ display: none;
+}
+
+#content.edge #edge-subscription
+{
+ display: block;
+}
+
#summary h1, #summary h2
{
font-weight: normal;
@@ -417,6 +427,16 @@
background-position: -128px -109px;
}
+#platform-edge
+{
+ background-position: -68px -171px;
+}
+
+#content.edge #platform-edge
+{
+ background-position: -68px -151px;
+}
+
.warning-platform,
.disclaimer,
#maxthon-notification
@@ -609,7 +629,8 @@
#more-container.internet-explorer #tab-internet-explorer,
#more-container.safari #tab-safari,
#more-container.yandex-browser #tab-yandex-browser,
-#more-container.maxthon #tab-maxthon
+#more-container.maxthon #tab-maxthon,
+#more-container.edge #tab-edge
{
display: inline-block;
}
@@ -622,6 +643,86 @@
#more-container.safari #more-safari,
#more-container.yandex-browser #more-yandex-browser,
#more-container.maxthon #more-maxthon
+#more-container.edge #more-edge
{
background-color: #dbd4c7;
}
+#content h3.subscribe-header
+{
+ font-size: 18px;
+ margin-bottom: 0px;
+}
+
+#edge-subscription h3
+{
+ margin: 19px 0px 10px 0px;
+ color: #2d79c0;
+ font-weight: bold;
+ font-family: Arial;
+ font-size: 14px;
+}
+
+#edge-subscription
+{
+ margin: 0px 0px 10px 40px;
saroyanm 2015/09/23 12:44:21 In general I'd be fan of not aligning this with th
+}
+
+#other-browsers
+{
+ display: none;
+}
+
+#subscribe-textbox
+{
+ border: solid 2px #a5a5a5;
+ margin-right: 0px;
+ padding-left: 10px;
saroyanm 2015/09/23 12:44:22 Please use padding: 0px 10px; here to make it cons
+ padding-right: 10px;
+ font-size: 14px;
+ font-family: Arial;
+ line-height: 36px;
+ height: 36px; /* IE 8 */
+}
+
+#subscribe-textbox.placeholder
+{
+ color: #a5a5a5;
+}
+
+#subscribe-button
+{
+ border: solid 1px #357EBD;
+ border-radius: 3px;
+ background-color: #428BCA;
+ color: #FFFFFF;
+ font-size: 14px;
+ font-weight: bold;
+ min-width: 146px;
saroyanm 2015/09/23 12:44:21 Please use width instead of min-width, so we will
+ height: 40px;
+ margin-left: 0px;
+}
+
+#subscribe-button span
saroyanm 2015/09/23 12:44:22 Please remove this rule, we should also consider,
+{
+ padding-left: 30px;
+ padding-right: 30px;
+}
+
+#subscribe-form .sucess-label, #subscribe-form .response-error,
+#subscribe-form .invalid-label, #subscribe-form.success input,
+#subscribe-form.success button
saroyanm 2015/09/23 12:44:21 Probably can be merged with rule above (#other-bro
+{
+ display: none;
+}
+
+#subscribe-form .invalid-label
+{
+ margin-top: 15px;
+}
+
+#subscribe-form.success .sucess-label,
+#subscribe-form.invalid .invalid-label,
+#subscribe-form.error .response-error
+{
+ display: block;
+}

Powered by Google App Engine
This is Rietveld