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

Issue 6192536888541184: Issue 1989 - Used icons with more details for 48px and 128px (Closed)

Created:
Feb. 13, 2015, 11:26 a.m. by Sebastian Noack
Modified:
Feb. 13, 2015, 12:14 p.m.
Reviewers:
kzar
CC:
Wladimir Palant, sven
Visibility:
Public.

Description

Issue 1989 - Used icons with more details for 48px and 128px

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -0 lines) Patch
M chrome/icons/abp-48.png View Binary file 0 comments Download
M icons/abp-128.png View Binary file 0 comments Download

Messages

Total messages: 4
Sebastian Noack
Feb. 13, 2015, 11:27 a.m. (2015-02-13 11:27:29 UTC) #1
kzar
The 48px icon looks OK, can't seem to get the 128px icon to render. The ...
Feb. 13, 2015, 11:45 a.m. (2015-02-13 11:45:41 UTC) #2
Sebastian Noack
I went ahead and pushed the change: https://hg.adblockplus.org/adblockpluschrome/rev/67a6a0082120 Feel free to double check the icons ...
Feb. 13, 2015, 11:54 a.m. (2015-02-13 11:54:26 UTC) #3
kzar
Feb. 13, 2015, 12:14 p.m. (2015-02-13 12:14:30 UTC) #4

Powered by Google App Engine
This is Rietveld