Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(129)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 years ago by Thomas Greiner
Modified:
5 years ago
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 ...
5 years ago (2015-02-03 13:12:43 UTC) #1
saroyanm
5 years ago (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"
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 87257f5