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

Side by Side 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.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | static/scss/main.scss » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 title=Utilities 1 title=Utilities
2 description=HTML & SCSS component utilities 2 description=HTML & SCSS component utilities
3 3
4 <h1>Utilities</h1> 4 <h1>Utilities</h1>
5 <p>HTML & SCSS component utilities</p> 5 <p>HTML & SCSS component utilities</p>
6 6
7 <ol> 7 <ol>
8 <li><a href="#backgrounds">Backgrounds</a></li> 8 <li><a href="#backgrounds">Backgrounds</a></li>
9 <li><a href="#text-alignment">Text Alignment</a></li> 9 <li><a href="#text-alignment">Text Alignment</a></li>
10 <li><a href="#lead-text">Lead text</a></li> 10 <li><a href="#lead-text">Lead text</a></li>
11 <li><a href="#horizontal-lists">Horizontal lists</a></li>
11 </ol> 12 </ol>
12 13
13 <h2 id="backgrounds">Backgrounds</h2> 14 <h2 id="backgrounds">Backgrounds</h2>
14 15
15 <p class="bg-primary">Primary background</p> 16 <p class="bg-primary">Primary background</p>
16 17
17 <p class="bg-secondary">Secondary background</p> 18 <p class="bg-secondary">Secondary background</p>
18 19
19 <p class="bg-accent">Accent background</p> 20 <p class="bg-accent">Accent background</p>
20 21
(...skipping 10 matching lines...) Expand all
31 32
32 <p class="text-right">Text aligned right</p> 33 <p class="text-right">Text aligned right</p>
33 34
34 <p class="text-end">Text aligned to the end of the container</p> 35 <p class="text-end">Text aligned to the end of the container</p>
35 36
36 <h2 id="lead-text">Lead text</h2> 37 <h2 id="lead-text">Lead text</h2>
37 38
38 <p class="lead"> "Lead text" is introductory text that is not part of a heading. </p> 39 <p class="lead"> "Lead text" is introductory text that is not part of a heading. </p>
39 40
40 <p>It is typically visually emphasized, not semantically emphasized, like <code> strong</code> or <code>em</code>.</p> 41 <p>It is typically visually emphasized, not semantically emphasized, like <code> strong</code> or <code>em</code>.</p>
42
43
44 <h2 id="horizontal-lists">Horizontal lists</h2>
45
46 <ul class="horizontal-list">
47 <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
48 <li>List item 2</li>
49 <li>List item 3</li>
50 <li>List item 4</li>
51 </ul>
OLDNEW
« 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