Issue 5135 - Reduce font size and padding on smaller screens on acceptableads.com

static/scss/content/_typography.scss

 // This file is part of acceptableads.org.
 // Copyright (C) 2016 Eyeo GmbH
 //
 // acceptableads.org is free software: you can redistribute it and/or modify
 // it under the terms of the GNU General Public License as published by
 // the Free Software Foundation, either version 3 of the License, or
 // (at your option) any later version.
 //
 // acceptableads.org is distributed in the hope that it will be useful,
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 13 matching lines @@
 }
 
 h1,
 .h1
 {
   margin: $xl 0 $lg 0;
   text-transform: none;
   font-size: $font-size-h1;
   font-weight: 300;
 
-  @media screen and (max-width: $tablet-breakpoint)
-  {
-    font-size: $font-size-h1 - 10px;
-    margin: $md 0 $lg 0;

[juliandoucette, 2017/05/30 14:22:57]

NIT: It would be more straightforward if you just changed margin-top

[ire, 2017/05/30 17:44:59]

Acknowledged.

+  @media (max-width: $tablet-breakpoint)
+  {
+    font-size: $font-size-h1 - 12px;
+    margin-top: $md;
+  }
+  @media (max-width: $mobile-breakpoint)
+  {
+    font-size: $font-size-h1 - 16px;
+    margin-bottom: $lg - 20px;
   }
 }
 
 h2,
 .h2
 {
   margin: $lg 0 $md 0;
   font-size: $font-size-h2;
   font-weight: 700;
 
-  @media screen and (max-width: $tablet-breakpoint)
-  {
-    margin: $md 0 $md 0;
+  @media (max-width: $tablet-breakpoint)
+  {
+    font-size: $font-size-h2 - 4px;
+    margin-top: $md;
+  }
+  @media (max-width: $mobile-breakpoint)
+  {
+    font-size: $font-size-h2 - 6px;
+    margin-bottom: $sm;
   }
 }
 
 h3,
 .h3
 {
   margin: $lg 0 $md 0;
   font-size: $font-size-h3;
   font-weight: 700;
+
+  @media (max-width: $tablet-breakpoint)
+  {
+    font-size: $font-size-h3 - 2px;
+    margin-top: $md;
+  }
+  @media (max-width: $mobile-breakpoint)
+  {
+    margin-bottom: $sm;
+  }
 }
 
 h4,
 .h4
 {
   margin: $md 0 $sm 0;
   font-size: $font-size-h4;
   font-weight: 700;
+
+  @media (max-width: $mobile-breakpoint)
+  {
+    font-size: $font-size-h4 - 2px;
+  }
 }
 
 h5,
 .h5
 {
   margin: $sm 0;
   font-size: $font-size-h5;
+
+  @media (max-width: $mobile-breakpoint)
+  {
+    font-size: $font-size-h5 - 2px;
+  }
 }
 
 h6,
 .h6
 {
   margin: $sm 0;
   font-size: $font-size-h6;
   font-weight: 400;
+
+  @media (max-width: $mobile-breakpoint)
+  {
+    font-size: $font-size-h6 - 2px;
+  }
 }
 
 // Main text ///////////////////////////////////////////////////////////////////
 
 html,
 body
 {
   color: $primary-fg;
   // @see https://www.smashingmagazine.com/2015/11/using-system-ui-fonts-practical-guide/
   font-family: "Source Sans Pro", -apple-system, BlinkMacSystemFont,
   "Segoe UI", "Roboto", "Oxygen", "Ubuntu", "Cantarell",
   "Fira Sans", "Droid Sans", "Helvetica Neue",
   sans-serif;
   font-size: $font-size-lg;
   font-weight: 300;
   line-height: 120%;
+
+  @media (max-width: $mobile-breakpoint)
+  {
+    font-size: $font-size-lg - 2px;
+  }
 }
 
 small,
 .small
 {
   margin-bottom: $xs;
   font-size: 70%;
   line-height: 80%;
 }
 
@@ skipping 154 matching lines @@
 dd
 {
   margin: $sm 0 $md 0;
 }
 
 dt
 {
   font-size: $font-size-xl;
   text-transform: uppercase;
   font-weight: 700;
+
+  @media (max-width: $tablet-breakpoint)
+  {
+    font-size: $font-size-xl - 2px;
+  }
 }
 
 // Blockquotes /////////////////////////////////////////////////////////////////
 
 blockquote
 {
   margin: 0;
   padding: 0 $md;
   text-transform: uppercase;
   font-size: $font-size-lg;
@@ skipping 16 matching lines @@
   border: none;
 }
 
 .center blockquote
 {
   @extend h2;
   margin: 0;
   padding: $lg 0;
   font-style: normal;
 
-  @media screen and (max-width: $tablet-breakpoint)
+  @media (max-width: $tablet-breakpoint)
   {
     font-size: $font-size-h3;
     padding: $md 0;
   }
 }
 
 .center blockquote p:last-child
 {
   margin-top: -$xs;
 }
@@ skipping 59 matching lines @@
   // hr background color is the line color
   background-color: $primary-fg;
 }
 
 // Place hr under h1 or h2 inside hN margin
 h1 + hr,
 h2 + hr
 {
   position: absolute;
   margin: -$md 0 $md 0;
+
+  @media (max-width: $mobile-breakpoint) 
+  {
+    margin-top: -$sm - 5px;

[ire, 2017/07/06 10:31:38]

The $xs size was every so slightly off

+  }
 }
 
 // center hr in centered blocks
 .center hr
 {
   margin: $sm auto;
 }
 
 // Tables //////////////////////////////////////////////////////////////////////
 
@@ skipping 33 matching lines @@
 {
   vertical-align: bottom;
   text-transform: uppercase;
   border-bottom: 2px solid $info;
 }
 
 table tbody + tbody
 {
   border-top: 2px solid $info;
 }