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

Issue 8677007: Hide raw filter editing link (Closed)

Created:
Oct. 24, 2012, 7:33 a.m. by Felix Dahlke
Modified:
Nov. 14, 2012, 7:12 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

I have tried to fix raw filter editing before giving Till a version to send to Opera, but couldn't make it work. It's the mysterious "Object it not a constructor" error message. So I've just hidden the link for now.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M options.html View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
Felix Dahlke
Oct. 24, 2012, 7:47 a.m. (2012-10-24 07:47:37 UTC) #1
Wladimir Palant
NOT LGTM, see http://codereview.adblockplus.org/8679008/.
Oct. 24, 2012, 9:04 a.m. (2012-10-24 09:04:01 UTC) #2
Felix Dahlke
Oct. 24, 2012, 10:51 a.m. (2012-10-24 10:51:40 UTC) #3
On 2012/10/24 09:04:01, Wladimir Palant wrote:
> NOT LGTM, see http://codereview.adblockplus.org/8679008/.

Closing, unmerged.

Powered by Google App Engine
This is Rietveld