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

Unified Diff: include.postload.js

Issue 6308777468887040: Fixed: Background color not restored when context menu clickhide canceled (Closed)
Patch Set: Created Nov. 28, 2013, 11:45 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
@@ -297,6 +297,10 @@
// Show popup
clickHide_showDialog(e.clientX, e.clientY, clickHideFilters);
+ // Set background color in case it got selected using context menu
+ if (typeof currentElement_backgroundColor == "undefined")
Wladimir Palant 2013/12/18 11:57:05 Given that we never reset currentElement_backgroun
Thomas Greiner 2013/12/18 12:33:26 Done. http://codereview.adblockplus.org/5661044793
+ currentElement_backgroundColor = currentElement.style.backgroundColor;
+
// Highlight the unlucky elements
// Restore currentElement's box-shadow and bgcolor so that highlightElements won't save those
currentElement.style.setProperty("-webkit-box-shadow", currentElement_boxShadow);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld