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

Unified Diff: static/css/index-desktop.css

Issue 29322769: Issue 2844 - Create Microsoft Edge coming soon landing page (Closed)
Patch Set: Refactor according to the style guide. Created Aug. 13, 2015, 2:41 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-desktop.css
===================================================================
--- a/static/css/index-desktop.css
+++ b/static/css/index-desktop.css
@@ -17,6 +17,11 @@
margin-bottom: 4px;
}
+#content.edge #main
+{
+ height: 380px;
+}
+
html[dir="rtl"] #main
{
float: left;
@@ -47,13 +52,25 @@
width: 115px;
}
-#content.maxthon #install ul
+#content.maxthon #install ul,
+#content.edge #install ul
{
margin-left: 18px;
right: auto;
width: auto;
}
+#content.edge #install ul
+{
+ margin: 0px 35px;
saroyanm 2015/08/20 20:54:54 Looking to visual seems like we have top margin he
Oleksandr 2015/08/26 00:19:29 Not sure what you mean here. The element positioni
+}
+
+#content.edge #edge-other-browsers
+{
+ margin: 0px 40px;
+ display: block;
+}
+
#information-android
{
left: 2px;
@@ -71,7 +88,7 @@
#adblock-browser-notification
{
- margin-top: 50px;
+ margin-top: 40px;
saroyanm 2015/08/20 20:54:54 Why this have been changed ?
Oleksandr 2015/08/26 00:19:29 It was to accommodate the style guide. But I don't
}
#key-facts
@@ -168,3 +185,14 @@
{
width: 760px;
}
+
+#content.edge #edge-incoming
+{
+ display: inline-block;
+ padding: 0px;
+}
+
+#subscribe-textbox
+{
+ width: 210px;
+}
« includes/index.tmpl ('K') | « static/css/index.css ('k') | static/css/index-mobile.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld