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

Side by Side 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.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 #content 1 #content
2 { 2 {
3 width: 990px; 3 width: 990px;
4 margin-bottom: 290px; 4 margin-bottom: 290px;
5 padding-left: 0; 5 padding-left: 0;
6 padding-right: 0; 6 padding-right: 0;
7 } 7 }
8 8
9 #main 9 #main
10 { 10 {
11 width: 426px; 11 width: 426px;
12 height: 307px; 12 height: 307px;
13 position: relative; 13 position: relative;
14 float: right; 14 float: right;
15 margin-left: 0px; 15 margin-left: 0px;
16 margin-right: 3px; 16 margin-right: 3px;
17 margin-bottom: 4px; 17 margin-bottom: 4px;
18 } 18 }
19 19
20 #content.edge #main
21 {
22 height: 380px;
23 }
24
20 html[dir="rtl"] #main 25 html[dir="rtl"] #main
21 { 26 {
22 float: left; 27 float: left;
23 margin-right: 0px; 28 margin-right: 0px;
24 margin-left: 8px; 29 margin-left: 8px;
25 } 30 }
26 31
27 #install .install-button, #install ul, #information-android 32 #install .install-button, #install ul, #information-android
28 { 33 {
29 position: absolute; 34 position: absolute;
(...skipping 10 matching lines...) Expand all
40 bottom: 0px; 45 bottom: 0px;
41 } 46 }
42 47
43 #install ul 48 #install ul
44 { 49 {
45 bottom: -2px; 50 bottom: -2px;
46 right: 0; 51 right: 0;
47 width: 115px; 52 width: 115px;
48 } 53 }
49 54
50 #content.maxthon #install ul 55 #content.maxthon #install ul,
56 #content.edge #install ul
51 { 57 {
52 margin-left: 18px; 58 margin-left: 18px;
53 right: auto; 59 right: auto;
54 width: auto; 60 width: auto;
55 } 61 }
56 62
63 #content.edge #install ul
64 {
65 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
66 }
67
68 #content.edge #edge-other-browsers
69 {
70 margin: 0px 40px;
71 display: block;
72 }
73
57 #information-android 74 #information-android
58 { 75 {
59 left: 2px; 76 left: 2px;
60 bottom: -25px; 77 bottom: -25px;
61 } 78 }
62 79
63 .warning-platform, .disclaimer, 80 .warning-platform, .disclaimer,
64 #maxthon-notification, #adblock-browser-notification 81 #maxthon-notification, #adblock-browser-notification
65 { 82 {
66 width: 100%; 83 width: 100%;
67 margin-top: 0; 84 margin-top: 0;
68 margin-bottom: 50px; 85 margin-bottom: 50px;
69 float: right; 86 float: right;
70 } 87 }
71 88
72 #adblock-browser-notification 89 #adblock-browser-notification
73 { 90 {
74 margin-top: 50px; 91 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
75 } 92 }
76 93
77 #key-facts 94 #key-facts
78 { 95 {
79 margin-top: 50px; 96 margin-top: 50px;
80 margin-left: 80px; 97 margin-left: 80px;
81 margin-right: 0px; 98 margin-right: 0px;
82 } 99 }
83 100
84 html[dir="rtl"] #key-facts 101 html[dir="rtl"] #key-facts
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 178
162 #more-tabs #sidebar button 179 #more-tabs #sidebar button
163 { 180 {
164 display: block; 181 display: block;
165 } 182 }
166 183
167 .more-tab 184 .more-tab
168 { 185 {
169 width: 760px; 186 width: 760px;
170 } 187 }
188
189 #content.edge #edge-incoming
190 {
191 display: inline-block;
192 padding: 0px;
193 }
194
195 #subscribe-textbox
196 {
197 width: 210px;
198 }
OLDNEW
« 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