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

Issue 29324587: Issue 2948 - Online Trust Alliance logo on press page (Closed)

Created:
Aug. 25, 2015, 3:07 p.m. by saroyanm
Modified:
Aug. 31, 2015, 9:54 a.m.
Reviewers:
Thomas Greiner
CC:
Felix Dahlke
Visibility:
Public.

Description

Issue 2948 - Online Trust Alliance logo on press page

Patch Set 1 #

Total comments: 4

Patch Set 2 : Changed logo placement/size and added link #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M includes/press/contact.md View 1 1 chunk +3 lines, -1 line 0 comments Download
A static/images/ota-logo.png View 1 Binary file 0 comments Download

Messages

Total messages: 4
saroyanm
@Thomas can you please have a look ? The OTA logo has been proceeded through ...
Aug. 25, 2015, 3:08 p.m. (2015-08-25 15:08:12 UTC) #1
Thomas Greiner
https://codereview.adblockplus.org/29324587/diff/29324588/includes/press/contact.md File includes/press/contact.md (right): https://codereview.adblockplus.org/29324587/diff/29324588/includes/press/contact.md#newcode3 includes/press/contact.md:3: ![Online Trust Alliance logo](/images/ota-logo.png) According to the ticket description ...
Aug. 25, 2015, 3:12 p.m. (2015-08-25 15:12:49 UTC) #2
saroyanm
https://codereview.adblockplus.org/29324587/diff/29324588/includes/press/contact.md File includes/press/contact.md (right): https://codereview.adblockplus.org/29324587/diff/29324588/includes/press/contact.md#newcode3 includes/press/contact.md:3: ![Online Trust Alliance logo](/images/ota-logo.png) On 2015/08/25 15:12:49, Thomas Greiner ...
Aug. 25, 2015, 3:33 p.m. (2015-08-25 15:33:12 UTC) #3
Thomas Greiner
Aug. 25, 2015, 3:46 p.m. (2015-08-25 15:46:49 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld