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

Issue 29723586: Issue 6484 - Add horizontal-list class to website-defaults (Closed)

Created:
March 15, 2018, 9:37 a.m. by ire
Modified:
April 3, 2018, 3:20 p.m.
Reviewers:
juliandoucette
Base URL:
https://hg.adblockplus.org/website-defaults
Visibility:
Public.

Description

Issue 6484 - Add horizontal-list class to website-defaults

Patch Set 1 #

Total comments: 4

Patch Set 2 : Addressed comments #2 #

Total comments: 2

Patch Set 3 : Add comma-separated class #

Patch Set 4 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+68 lines, -0 lines) Patch
M pages/utilities.html View 1 2 2 chunks +20 lines, -0 lines 0 comments Download
M static/scss/main.scss View 1 2 3 1 chunk +1 line, -0 lines 0 comments Download
A static/scss/utilities/_lists.scss View 1 2 1 chunk +47 lines, -0 lines 0 comments Download

Messages

Total messages: 7
ire
March 15, 2018, 9:37 a.m. (2018-03-15 09:37:12 UTC) #1
juliandoucette
No example page? https://codereview.adblockplus.org/29723586/diff/29723587/static/scss/utilities/_lists.scss File static/scss/utilities/_lists.scss (right): https://codereview.adblockplus.org/29723586/diff/29723587/static/scss/utilities/_lists.scss#newcode31 static/scss/utilities/_lists.scss:31: .content .horizontal-list, I think it's OK ...
March 24, 2018, 3:48 p.m. (2018-03-24 15:48:09 UTC) #2
ire
On 2018/03/24 15:48:09, juliandoucette wrote: > No example page? Added to the utilities page. https://codereview.adblockplus.org/29723586/diff/29723587/static/scss/utilities/_lists.scss ...
March 26, 2018, 6:59 a.m. (2018-03-26 06:59:39 UTC) #3
juliandoucette
LGTM + NIT https://codereview.adblockplus.org/29723586/diff/29733595/pages/utilities.html File pages/utilities.html (right): https://codereview.adblockplus.org/29723586/diff/29733595/pages/utilities.html#newcode47 pages/utilities.html:47: <li>List item 1</li> NIT: This could ...
April 3, 2018, 1:35 p.m. (2018-04-03 13:35:13 UTC) #4
ire
https://codereview.adblockplus.org/29723586/diff/29733595/pages/utilities.html File pages/utilities.html (right): https://codereview.adblockplus.org/29723586/diff/29733595/pages/utilities.html#newcode47 pages/utilities.html:47: <li>List item 1</li> On 2018/04/03 13:35:13, juliandoucette wrote: > ...
April 3, 2018, 2:36 p.m. (2018-04-03 14:36:34 UTC) #5
ire
New patch set up
April 3, 2018, 2:41 p.m. (2018-04-03 14:41:58 UTC) #6
juliandoucette
April 3, 2018, 2:51 p.m. (2018-04-03 14:51:38 UTC) #7
LGTM

Powered by Google App Engine
This is Rietveld