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

Unified Diff: include.postload.js

Issue 6194342989398016: Issue 2298 - Fixed: "Block element" from context menu doesn't reset previous state (Closed)
Patch Set: Created April 10, 2015, 11:07 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include.postload.js
===================================================================
--- a/include.postload.js
+++ b/include.postload.js
@@ -357,9 +357,7 @@
function clickHide_deactivate()
{
clickHide_rulesPending();
-
clickHide_filters = null;
- lastRightClickEvent = null;
Sebastian Noack 2015/04/10 11:11:17 In case you wonder, resetting the lastRightClickEv
Sebastian Noack 2015/04/10 11:32:49 Never mind. I reverted my decision. Cleaning up ma
if (currentElement)
{
@@ -699,7 +697,7 @@
{
getBlockableElementOrAncestor(lastRightClickEvent.target, function(element)
{
- clickHide_activated = true;
+ clickHide_activate();
currentElement = element;
clickHide_mouseClick(lastRightClickEvent);
});
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld