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

Unified Diff: pages/about/best-practices.md

Issue 29438582: Issue 5135 - Reduce font size and padding on smaller screens on acceptableads.com (Closed) Base URL: https://hg.adblockplus.org/web.acceptableads.com
Patch Set: Created May 16, 2017, 4:26 a.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 | pages/about/index.html » ('j') | static/scss/components/_cards.scss » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pages/about/best-practices.md
===================================================================
--- a/pages/about/best-practices.md
+++ b/pages/about/best-practices.md
@@ -10,17 +10,17 @@
---
{{best-practices-summary[Best practices summary] The following guidelines have been taken from successful cases of websites implementing Acceptable Ads. Make your website look awesome and start monetizing good forms of advertising, too. It’s this easy:}}
</div>
</div>
-<div class="bg-accent m-y-xl" markdown="1">
+<div class="bg-accent section" markdown="1">
juliandoucette 2017/05/16 19:07:34 Is there a reason why you put section here instead
ire 2017/05/16 21:32:27 Yes. So this seemed to fit the description/styling
juliandoucette 2017/05/17 13:09:11 Acknowledged.
<div class="container center" markdown="1">
> {{best-practices-quote-1[Best practices summary quote line 1] Choose the best formats in the best spots.}}<br> {{best-practices-quote-2[Best practices summary quote line 2] Be clear and relevant. It’s really that simple.}}
</div>
</div>
<div class="container col-6 center" markdown="1">
« no previous file with comments | « no previous file | pages/about/index.html » ('j') | static/scss/components/_cards.scss » ('J')

Powered by Google App Engine
This is Rietveld