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: Rebased to the current tip. Created Sept. 21, 2015, 11:10 p.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
saroyanm 2015/09/22 13:15:45 Should be "#edge-teaser"
{
background: #ffffff;
padding: 10px;
@@ -96,6 +96,27 @@
display: inline-block;
}
+#feature-edge, #edge-teaser
saroyanm 2015/09/22 13:15:46 This rule and " #content.edge #video" can be merge
+{
+ display: none;
+}
+
+#edge-teaser
saroyanm 2015/09/22 13:15:46 This block is redundant while the image itself is
+{
+ width: 540;
+ height: 305;
+}
+
+#content.edge #video
+{
+ display: none;
+}
+
+#content.edge #feature-edge
+{
+ display: block;
+}
+
#summary h1, #summary h2
{
font-weight: normal;
@@ -124,6 +145,11 @@
color: #333333;
}
+#content.edge #summary ul
saroyanm 2015/09/22 13:15:46 I assume after you will move subscription block ou
+{
+ margin: 10px 0px 18px 0px;
+}
+
html[dir="rtl"] #summary ul
{
padding-right: 40px;
@@ -417,6 +443,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 +645,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 +659,81 @@
#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;
+}
+
+.subscribe-description
+{
+ margin: 19px 0px 10px 0px;
+ color: #2d79c0;
+ font-weight: bold;
+ font-family: Arial;
+ font-size: 14px;
+}
+
+#other-browsers
+{
+ display: none;
+}
+
+#subscribe-textbox
+{
+ border: solid 2px #a5a5a5;
+ margin-right: 0px;
+ padding-left: 10px;
+ padding-right: 10px;
+ font-size: 14px;
+ font-family: Arial;
+ line-height: 36px;
+ height: 36px; /* IE 8 */
saroyanm 2015/09/22 13:15:46 This should be redundant if we don't expect IE8 us
Oleksandr 2015/09/23 00:37:57 We still support IE8. It's the oldest IE we suppor
saroyanm 2015/09/23 12:44:20 Ahh fare enough, don't know why I thought we would
+}
+
+#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;
+ height: 40px;
+ margin-left: 0px;
+}
+
+#subscribe-button span
+{
+ padding-left: 30px;
+ padding-right: 30px;
+}
+
+#subscribe-form .sucess-label, #response-error,
+#subscribe-form .invalid-label, #subscribe-form.success input,
+#subscribe-form.success button
+{
+ 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