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

Issue 5640105578463232: Issue 1905 - Added Stack Exchange as supporter to Manifesto page (Closed)

Created:
Feb. 3, 2015, 1:06 p.m. by Thomas Greiner
Modified:
Feb. 9, 2015, 10:57 a.m.
Reviewers:
saroyanm
CC:
Felix Dahlke
Visibility:
Public.

Description

Issue 1905 - Added Stack Exchange as supporter to Manifesto page

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -18 lines) Patch
M acceptable-ads-manifesto_body.html View 1 chunk +22 lines, -18 lines 1 comment Download
A stackexchange.png View Binary file 0 comments Download

Messages

Total messages: 2
Thomas Greiner
As mentioned in #1905 this is not only adding Stack Exchange to the list of ...
Feb. 3, 2015, 1:12 p.m. (2015-02-03 13:12:43 UTC) #1
saroyanm
Feb. 5, 2015, 2:56 p.m. (2015-02-05 14:56:20 UTC) #2
LGTM with small fix.

http://codereview.adblockplus.org/5640105578463232/diff/5629499534213120/acce...
File acceptable-ads-manifesto_body.html (right):

http://codereview.adblockplus.org/5640105578463232/diff/5629499534213120/acce...
acceptable-ads-manifesto_body.html:87: <img src="images/manifesto/pagefair.png"
alt="Page Fair"/>
Please change alt="Page Fair" to alt="PageFair"

Powered by Google App Engine
This is Rietveld