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

Unified Diff: static/css/main.css

Issue 29581842: Noissue - Refactored new abp.org mobile heading sizes (Closed) Base URL: https://bitbucket.org/adblockplus/adblockplus.org
Patch Set: Created Oct. 17, 2017, 2:53 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: static/css/main.css
===================================================================
--- a/static/css/main.css
+++ b/static/css/main.css
@@ -153,22 +153,28 @@
.content h4,
.content h5
{
font-weight: bold;
margin: 32px 0px;
margin: 2rem 0rem;
}
-.content h1 { font-size: 2.4em; }
-.content h2 { font-size: 1.6em; }
+.content h1 { font-size: 1.8em; }
+.content h2 { font-size: 1.4em; }
.content h3 { font-size: 1.3em; }
.content h4 { font-size: 1.2em; }
.content h5 { font-size: 1.1em; }
+@media(min-width: 1200px)
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
+{
+ .content h1 { font-size: 2.4em; }
+ .content h2 { font-size: 1.6em; }
+}
+
/*******************************************************************************
* #navbar
******************************************************************************/
#navbar
{
height: 4em;
background-color: #c70d2c;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld