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

Issue 5086035738361856: Issue 1534 - Replace ABP icon with new one (Closed)

Created:
Feb. 25, 2015, 5:18 p.m. by Wladimir Palant
Modified:
Feb. 25, 2015, 7:23 p.m.
Reviewers:
Thomas Greiner
Visibility:
Public.

Description

Issue 1534 - Replace ABP icon with new one

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -0 lines) Patch
M chrome/skin/abp-status.png View Binary file 0 comments Download
M chrome/skin/abp-status-16.png View Binary file 0 comments Download
M chrome/skin/abp-status-32.png View Binary file 0 comments Download
M icon.png View Binary file 0 comments Download
M icon64.png View Binary file 0 comments Download

Messages

Total messages: 3
Wladimir Palant
Feb. 25, 2015, 5:18 p.m. (2015-02-25 17:18:31 UTC) #1
Thomas Greiner
LGTM for this "code" review :)
Feb. 25, 2015, 7:06 p.m. (2015-02-25 19:06:16 UTC) #2
Wladimir Palant
Feb. 25, 2015, 7:22 p.m. (2015-02-25 19:22:06 UTC) #3
On 2015/02/25 19:06:16, Thomas Greiner wrote:
> LGTM for this "code" review :)

You'd be surprised how much can go wrong when replacing images ;)

Powered by Google App Engine
This is Rietveld