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

Issue 29324393: Issue 2928 - Add favicon [acceptableads.org] (Closed)

Created:
Aug. 19, 2015, 4:26 p.m. by saroyanm
Modified:
Aug. 20, 2015, 2:54 p.m.
Reviewers:
Thomas Greiner
CC:
Felix Dahlke
Visibility:
Public.

Description

Issue 2928 - Add favicon [acceptableads.org]

Patch Set 1 #

Total comments: 2

Patch Set 2 : Made consistent with ABP.org #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M pages/index.html View 1 1 chunk +1 line, -0 lines 0 comments Download
A static/images/favicon-large.png View 1 Binary file 0 comments Download

Messages

Total messages: 4
saroyanm
Thomas can you also review this, similar to previous review. Aligned with Sven we need ...
Aug. 19, 2015, 4:28 p.m. (2015-08-19 16:28:59 UTC) #1
Thomas Greiner
https://codereview.adblockplus.org/29324393/diff/29324394/pages/index.html File pages/index.html (right): https://codereview.adblockplus.org/29324393/diff/29324394/pages/index.html#newcode12 pages/index.html:12: <link rel="icon" href="/images/favicon.png" /> In https://codereview.adblockplus.org/29324389/ you were using ...
Aug. 20, 2015, 10:08 a.m. (2015-08-20 10:08:54 UTC) #2
saroyanm
New patch uploaded https://codereview.adblockplus.org/29324393/diff/29324394/pages/index.html File pages/index.html (right): https://codereview.adblockplus.org/29324393/diff/29324394/pages/index.html#newcode12 pages/index.html:12: <link rel="icon" href="/images/favicon.png" /> On 2015/08/20 ...
Aug. 20, 2015, 12:30 p.m. (2015-08-20 12:30:19 UTC) #3
Thomas Greiner
Aug. 20, 2015, 2:46 p.m. (2015-08-20 14:46:00 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld