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

Issue 29565748: Issue 5834 - Removed learn more section from the adblockplus.org homepage (Closed)

Created:
Oct. 5, 2017, 3:50 p.m. by juliandoucette
Modified:
Oct. 6, 2017, 11:22 a.m.
Reviewers:
ire
Base URL:
https://bitbucket.org/adblockplus/adblockplus.org
Visibility:
Public.

Description

Issue 5834 - Removed learn more section from the adblockplus.org homepage

Patch Set 1 #

Total comments: 6

Patch Set 2 : Removed more learn more artifacts #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -344 lines) Patch
M includes/index.tmpl View 1 chunk +0 lines, -163 lines 0 comments Download
M static/css/index.css View 1 1 chunk +0 lines, -98 lines 0 comments Download
M static/css/index-desktop.css View 1 1 chunk +0 lines, -38 lines 0 comments Download
M static/css/index-mobile.css View 1 1 chunk +0 lines, -17 lines 0 comments Download
R static/js/index.js View 1 chunk +0 lines, -28 lines 0 comments Download

Messages

Total messages: 5
juliandoucette
Oct. 5, 2017, 3:50 p.m. (2017-10-05 15:50:36 UTC) #1
ire
Thanks Julian https://codereview.adblockplus.org/29565748/diff/29565749/static/css/index-desktop.css File static/css/index-desktop.css (right): https://codereview.adblockplus.org/29565748/diff/29565749/static/css/index-desktop.css#newcode173 static/css/index-desktop.css:173: #more-tabs #sidebar button #more-tabs has also been ...
Oct. 5, 2017, 8:26 p.m. (2017-10-05 20:26:48 UTC) #2
juliandoucette
Thanks Ire! Good catches. https://codereview.adblockplus.org/29565748/diff/29565749/static/css/index-desktop.css File static/css/index-desktop.css (right): https://codereview.adblockplus.org/29565748/diff/29565749/static/css/index-desktop.css#newcode173 static/css/index-desktop.css:173: #more-tabs #sidebar button On 2017/10/05 ...
Oct. 6, 2017, 10:21 a.m. (2017-10-06 10:21:11 UTC) #3
ire
On 2017/10/06 10:21:11, juliandoucette wrote: > Thanks Ire! > > Good catches. You're welcome :) ...
Oct. 6, 2017, 10:54 a.m. (2017-10-06 10:54:24 UTC) #4
juliandoucette
Oct. 6, 2017, 11:22 a.m. (2017-10-06 11:22:21 UTC) #5

Powered by Google App Engine
This is Rietveld