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

Unified Diff: include.postload.js

Issue 5661044793933824: Addressed comment on review 6308777468887040 (Closed)
Patch Set: Created Dec. 18, 2013, 12:22 p.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
@@ -316,10 +316,6 @@
// 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")
- 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);
@@ -554,6 +550,7 @@
// Coerce red highlighted overlay on top of element to remove.
// TODO: Wow, the design of the clickHide stuff is really dumb - gotta fix it sometime
currentElement = addElementOverlay(target);
+ currentElement_backgroundColor = target.style.backgroundColor;
// clickHide_mouseOver(lastRightClickEvent);
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