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

Delta Between Two Patch Sets: static/css/main.css

Issue 29581842: Noissue - Refactored new abp.org mobile heading sizes (Closed) Base URL: https://bitbucket.org/adblockplus/adblockplus.org
Left Patch Set: Created Oct. 17, 2017, 2:53 p.m.
Right Patch Set: Changed font-size breakpoint from large desktop to tablet Created Oct. 20, 2017, 2:07 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 | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 * 1 *
2 { 2 {
3 font-family: Arial, sans; 3 font-family: Arial, sans;
4 font-size: 16px; 4 font-size: 16px;
5 } 5 }
6 6
7 body 7 body
8 { 8 {
9 margin: 0; 9 margin: 0;
10 line-height: 1.5; 10 line-height: 1.5;
11 } 11 }
12 12
13 a img 13 a img
14 { 14 {
15 border: none; 15 border: none;
16 } 16 }
17 17
18 a:link, a:visited 18 a:link, a:visited
19 { 19 {
20 color: #555; 20 color: #555;
21 } 21 }
22 22
23 button 23 button
24 { 24 {
25 cursor: pointer; 25 cursor: pointer;
26 } 26 }
27 27
28 code, pre 28 code, pre
29 { 29 {
30 font-family: "Courier New", monospace; 30 font-family: monospace;
31 } 31 }
32 32
33 pre 33 pre
34 { 34 {
35 background: #fff8dc; 35 background: #ececec;
36 padding: 10px; 36 padding: 1em;
37 border: 1px solid black;
38 overflow: auto; 37 overflow: auto;
39 } 38 }
40 39
41 .sprite 40 .sprite
42 { 41 {
43 display: inline-block; 42 display: inline-block;
44 font-size: 0px; /* fix for IE6 height bug */ 43 font-size: 0px; /* fix for IE6 height bug */
45 background-image: url(../img/sprite-main.png); 44 background-image: url(../img/sprite-main.png);
46 background-repeat: no-repeat; 45 background-repeat: no-repeat;
47 } 46 }
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
157 margin: 32px 0px; 156 margin: 32px 0px;
158 margin: 2rem 0rem; 157 margin: 2rem 0rem;
159 } 158 }
160 159
161 .content h1 { font-size: 1.8em; } 160 .content h1 { font-size: 1.8em; }
162 .content h2 { font-size: 1.4em; } 161 .content h2 { font-size: 1.4em; }
163 .content h3 { font-size: 1.3em; } 162 .content h3 { font-size: 1.3em; }
164 .content h4 { font-size: 1.2em; } 163 .content h4 { font-size: 1.2em; }
165 .content h5 { font-size: 1.1em; } 164 .content h5 { font-size: 1.1em; }
166 165
167 @media(min-width: 1200px) 166 @media(min-width: 768px)
ire 2017/10/20 07:36:00 NIT: I think this is too wide? I think the larger
juliandoucette 2017/10/20 13:52:15 Agreed. We can use a variable for this purpose.
juliandoucette 2017/10/20 14:03:16 I was thinking about website-defaults :D
168 { 167 {
169 .content h1 { font-size: 2.4em; } 168 .content h1 { font-size: 2.4em; }
170 .content h2 { font-size: 1.6em; } 169 .content h2 { font-size: 1.6em; }
170 }
171
172 .content a,
173 .content a:visited
174 {
175 color: #c70d2c;
171 } 176 }
172 177
173 /******************************************************************************* 178 /*******************************************************************************
174 * #navbar 179 * #navbar
175 ******************************************************************************/ 180 ******************************************************************************/
176 181
177 #navbar 182 #navbar
178 { 183 {
179 height: 4em; 184 height: 4em;
180 background-color: #c70d2c; 185 background-color: #c70d2c;
(...skipping 421 matching lines...) Expand 10 before | Expand all | Expand 10 after
602 { 607 {
603 display: none; 608 display: none;
604 } 609 }
605 610
606 /* overriding display:none above */ 611 /* overriding display:none above */
607 #footer #social-list 612 #footer #social-list
608 { 613 {
609 display: block; 614 display: block;
610 } 615 }
611 } 616 }
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld