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

Side by Side Diff: scss/_content.scss

Issue 29551691: Noissue - Make website-default headings bold (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Created Sept. 21, 2017, 4:47 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /*! 1 /*!
2 * This file is part of website-defaults 2 * This file is part of website-defaults
3 * Copyright (C) 2016-present eyeo GmbH 3 * Copyright (C) 2016-present eyeo GmbH
4 * 4 *
5 * website-defaults is free software: you can redistribute it and/or 5 * website-defaults is free software: you can redistribute it and/or
6 * modify it under the terms of the GNU General Public License as published by 6 * modify it under the terms of the GNU General Public License as published by
7 * the Free Software Foundation, either version 3 of the License, or 7 * the Free Software Foundation, either version 3 of the License, or
8 * (at your option) any later version. 8 * (at your option) any later version.
9 * 9 *
10 * website-defaults is distributed in the hope that it will be useful, 10 * website-defaults is distributed in the hope that it will be useful,
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 46
47 h1, 47 h1,
48 h2, 48 h2,
49 h3, 49 h3,
50 h4, 50 h4,
51 h5, 51 h5,
52 h6 52 h6
53 { 53 {
54 /* Margin on top **only** (opinionated) */ 54 /* Margin on top **only** (opinionated) */
55 margin: 2em 0em 0.5em 0em; 55 margin: 2em 0em 0.5em 0em;
56 /* All headings should be bold (opinionated) */
57 font-weight: bold;
ire 2017/09/22 08:33:29 Why didn't you use the $bold-weight variable?
juliandoucette 2017/09/22 11:36:32 I forgot about it. Good catch.
juliandoucette 2017/09/22 11:38:30 Done.
56 } 58 }
57 59
58 h1 60 h1
59 { 61 {
60 font-size: 2em; 62 font-size: 2em;
61 } 63 }
62 64
63 h2 65 h2
64 { 66 {
65 font-size: 1.5em; 67 font-size: 1.5em;
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 { 161 {
160 /* undo browser default (opinionated)*/ 162 /* undo browser default (opinionated)*/
161 font-weight: $bold-weight; 163 font-weight: $bold-weight;
162 } 164 }
163 165
164 dd 166 dd
165 { 167 {
166 margin: 0.25em 0em 1em 0em; 168 margin: 0.25em 0em 1em 0em;
167 } 169 }
168 } 170 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld