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

Delta Between Two Patch Sets: static/css/index-desktop.css

Issue 29322769: Issue 2844 - Create Microsoft Edge coming soon landing page (Closed)
Left Patch Set: Refactor according to the style guide. Created Aug. 13, 2015, 2:41 a.m.
Right Patch Set: Use querySelectorAll instead of getElementsByClassName. Created Sept. 30, 2015, 7:56 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « static/css/index.css ('k') | static/css/index-mobile.css » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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 }
8
9 #content.ua-android,
10 #content.ua-ios
11 {
12 width: 1000px;
7 } 13 }
8 14
9 #main 15 #main
10 { 16 {
11 width: 426px; 17 width: 426px;
12 height: 307px; 18 height: 307px;
13 position: relative; 19 position: relative;
14 float: right; 20 float: right;
15 margin-left: 0px; 21 margin-left: 0px;
16 margin-right: 3px; 22 margin-right: 3px;
(...skipping 23 matching lines...) Expand all
40 left: 0; 46 left: 0;
41 } 47 }
42 48
43 #install .install-button:active 49 #install .install-button:active
44 { 50 {
45 bottom: 0px; 51 bottom: 0px;
46 } 52 }
47 53
48 #install ul 54 #install ul
49 { 55 {
50 bottom: -2px;
51 right: 0; 56 right: 0;
52 width: 115px; 57 width: 115px;
53 } 58 }
54 59
55 #content.maxthon #install ul, 60 #content.maxthon #install ul,
56 #content.edge #install ul 61 #content.edge #install ul
57 { 62 {
58 margin-left: 18px; 63 margin-left: 18px;
59 right: auto; 64 right: auto;
60 width: auto; 65 width: auto;
61 } 66 }
62 67
63 #content.edge #install ul 68 #content.edge #install ul
64 { 69 {
65 margin: 0px 35px; 70 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 } 71 }
67 72
68 #content.edge #edge-other-browsers 73 #content.edge #other-browsers
69 { 74 {
70 margin: 0px 40px; 75 margin: 0px 40px 10px 40px;
71 display: block; 76 display: block;
77 }
78
79 #content.edge #edge-teaser
80 {
81 display: inline-block;
82 padding: 0px;
83 }
84
85 #edge-subscription input[type=email]
86 {
87 width: 210px;
72 } 88 }
73 89
74 #information-android 90 #information-android
75 { 91 {
76 left: 2px; 92 left: 2px;
77 bottom: -25px; 93 bottom: -25px;
78 } 94 }
79 95
80 .warning-platform, .disclaimer, 96 #messages-wrapper
81 #maxthon-notification, #adblock-browser-notification 97 {
82 { 98 margin-top: 50px;
99 margin-bottom: 30px;
83 width: 100%; 100 width: 100%;
84 margin-top: 0;
85 margin-bottom: 50px;
86 float: right; 101 float: right;
87 } 102 }
88 103
89 #adblock-browser-notification 104 #messages-wrapper > div
90 { 105 {
91 margin-top: 40px; 106 margin-top: 0px;
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
107 margin-bottom: 20px;
92 } 108 }
93 109
94 #key-facts 110 #key-facts
95 { 111 {
96 margin-top: 50px; 112 margin-top: 50px;
97 margin-left: 80px; 113 margin-left: 80px;
98 margin-right: 0px; 114 margin-right: 0px;
99 } 115 }
100 116
101 html[dir="rtl"] #key-facts 117 html[dir="rtl"] #key-facts
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
177 } 193 }
178 194
179 #more-tabs #sidebar button 195 #more-tabs #sidebar button
180 { 196 {
181 display: block; 197 display: block;
182 } 198 }
183 199
184 .more-tab 200 .more-tab
185 { 201 {
186 width: 760px; 202 width: 760px;
187 } 203 }
188
189 #content.edge #edge-incoming
190 {
191 display: inline-block;
192 padding: 0px;
193 }
194
195 #subscribe-textbox
196 {
197 width: 210px;
198 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld