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

Issue 5071126564700160: Issue 1535 - Got rid of obsolete safari/icons/abp-32.png (Closed)

Created:
Jan. 27, 2015, 2:04 p.m. by Sebastian Noack
Modified:
Jan. 27, 2015, 4:16 p.m.
Reviewers:
kzar
CC:
Wladimir Palant
Visibility:
Public.

Description

This icon is redundant with icons/abp-32.png. Formerly we had two different versions of the 32x32px icon, one with less details shown in the toolbar icon on Safari, and one with more details for the options page and "Block element" dialog. However, we use the same icon everywhere now.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -19 lines) Patch
M metadata.safari View 1 chunk +19 lines, -19 lines 0 comments Download
R safari/icons/abp-32.png View Binary file 0 comments Download

Messages

Total messages: 2
Sebastian Noack
Jan. 27, 2015, 2:10 p.m. (2015-01-27 14:10:27 UTC) #1
kzar
Jan. 27, 2015, 2:27 p.m. (2015-01-27 14:27:43 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld