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

Unified Diff: static/scss/content/_typography.scss

Issue 29374562: Issue 4514 - Improved font readability on acceptableads.com (Closed) Base URL: https://hg.adblockplus.org/web.acceptableads.com
Patch Set: Created Feb. 4, 2017, 4:59 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 | « static/scss/_variables.scss ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: static/scss/content/_typography.scss
===================================================================
--- a/static/scss/content/_typography.scss
+++ b/static/scss/content/_typography.scss
@@ -15,17 +15,16 @@
// along with acceptableads.org. If not, see <http://www.gnu.org/licenses/>.
// Headings ////////////////////////////////////////////////////////////////////
%headings
{
text-transform: uppercase;
font-weight: 700;
- line-height: 120%;
}
h1,
.h1
{
margin: $xl 0 $lg 0;
text-transform: none;
font-size: $font-size-h1;
@@ -79,17 +78,17 @@
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%;
+ line-height: 1.5;
}
small,
.small
{
margin-bottom: $xs;
font-size: 70%;
line-height: 80%;
« no previous file with comments | « static/scss/_variables.scss ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld