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

Unified 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.
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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scss/_content.scss
===================================================================
--- a/scss/_content.scss
+++ b/scss/_content.scss
@@ -48,16 +48,18 @@
h2,
h3,
h4,
h5,
h6
{
/* Margin on top **only** (opinionated) */
margin: 2em 0em 0.5em 0em;
+ /* All headings should be bold (opinionated) */
+ 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.
}
h1
{
font-size: 2em;
}
h2
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld