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

Issue 6194342989398016: Issue 2298 - Fixed: "Block element" from context menu doesn't reset previous state (Closed)

Created:
April 10, 2015, 11:07 a.m. by Sebastian Noack
Modified:
April 13, 2015, 12:38 p.m.
Reviewers:
kzar
CC:
Wladimir Palant
Visibility:
Public.

Description

Issue 2298 - Fixed: "Block element" from context menu doesn't reset previous state

Patch Set 1 #

Total comments: 2

Patch Set 2 : Cleanup lastRightClickEvent #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -3 lines) Patch
M include.postload.js View 1 1 chunk +4 lines, -3 lines 0 comments Download

Messages

Total messages: 3
Sebastian Noack
http://codereview.adblockplus.org/6194342989398016/diff/5629499534213120/include.postload.js File include.postload.js (left): http://codereview.adblockplus.org/6194342989398016/diff/5629499534213120/include.postload.js#oldcode362 include.postload.js:362: lastRightClickEvent = null; In case you wonder, resetting the ...
April 10, 2015, 11:11 a.m. (2015-04-10 11:11:17 UTC) #1
Sebastian Noack
http://codereview.adblockplus.org/6194342989398016/diff/5629499534213120/include.postload.js File include.postload.js (left): http://codereview.adblockplus.org/6194342989398016/diff/5629499534213120/include.postload.js#oldcode362 include.postload.js:362: lastRightClickEvent = null; On 2015/04/10 11:11:17, Sebastian Noack wrote: ...
April 10, 2015, 11:32 a.m. (2015-04-10 11:32:49 UTC) #2
kzar
April 13, 2015, 10:53 a.m. (2015-04-13 10:53:42 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld