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

Unified Diff: pages/utilities.html

Issue 29723586: Issue 6484 - Add horizontal-list class to website-defaults (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Addressed comments #2 Created March 26, 2018, 6:56 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 | static/scss/main.scss » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pages/utilities.html
===================================================================
--- a/pages/utilities.html
+++ b/pages/utilities.html
@@ -3,16 +3,17 @@
<h1>Utilities</h1>
<p>HTML & SCSS component utilities</p>
<ol>
<li><a href="#backgrounds">Backgrounds</a></li>
<li><a href="#text-alignment">Text Alignment</a></li>
<li><a href="#lead-text">Lead text</a></li>
+ <li><a href="#horizontal-lists">Horizontal lists</a></li>
</ol>
<h2 id="backgrounds">Backgrounds</h2>
<p class="bg-primary">Primary background</p>
<p class="bg-secondary">Secondary background</p>
@@ -33,8 +34,18 @@
<p class="text-end">Text aligned to the end of the container</p>
<h2 id="lead-text">Lead text</h2>
<p class="lead"> "Lead text" is introductory text that is not part of a heading.</p>
<p>It is typically visually emphasized, not semantically emphasized, like <code>strong</code> or <code>em</code>.</p>
+
+
+<h2 id="horizontal-lists">Horizontal lists</h2>
+
+<ul class="horizontal-list">
+ <li>List item 1</li>
juliandoucette 2018/04/03 13:35:13 NIT: This could use commas and/or a little spacing
ire 2018/04/03 14:36:33 I think that could be an enhancement, e.g. a comma
+ <li>List item 2</li>
+ <li>List item 3</li>
+ <li>List item 4</li>
+</ul>
« no previous file with comments | « no previous file | static/scss/main.scss » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld