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

Issue 29527643: Fixed #5576 = Trash icon is missing from whitelist cell (Closed)

Created:
Aug. 25, 2017, 10:51 a.m. by a.shephard
Modified:
Aug. 29, 2017, 10:08 a.m.
Reviewers:
d108, dean, Felix Dahlke
Visibility:
Public.

Description

Fixed #5576 = Trash icon is missing from whitelist cell

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M AdblockPlusSafari/Base.lproj/Main.storyboard View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5
a.shephard
Aug. 25, 2017, 10:52 a.m. (2017-08-25 10:52:45 UTC) #1
dean
On 2017/08/25 10:52:45, a.shephard wrote: LGTM - Renders correctly from my tests.
Aug. 25, 2017, noon (2017-08-25 12:00:23 UTC) #2
d108
I've confirmed the fix of the trash icon, good one, but there is a patch ...
Aug. 28, 2017, 7:58 a.m. (2017-08-28 07:58:35 UTC) #3
a.shephard
On 2017/08/28 07:58:35, Daniel wrote: > I've confirmed the fix of the trash icon, good ...
Aug. 28, 2017, 8:54 a.m. (2017-08-28 08:54:28 UTC) #4
Felix Dahlke
Aug. 28, 2017, 4:18 p.m. (2017-08-28 16:18:49 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld