From 31d2f90bfeb5851dbef448ae30eac5782892452b Mon Sep 17 00:00:00 2001
From: Vince <grosbouddha@gmail.com>
Date: Sun, 16 Sep 2012 15:35:30 +0200
Subject: [PATCH] highlight target tool pixel - intial commit

---
 js/Constants.js             |  2 ++
 js/drawingtools/BaseTool.js | 19 ++++++++++++++++++-
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/js/Constants.js b/js/Constants.js
index 1f676f0e..c9440a27 100644
--- a/js/Constants.js
+++ b/js/Constants.js
@@ -8,6 +8,8 @@ var Constants = {
      * strokes and rectangles:
      */
     SELECTION_TRANSPARENT_COLOR: 'rgba(255, 255, 255, 0.6)',
+
+    TOOL_TARGET_HIGHLIGHT_COLOR: 'rgba(255, 255, 255, 0.2)',
     
     /*
      * Default entry point for piskel web service:
diff --git a/js/drawingtools/BaseTool.js b/js/drawingtools/BaseTool.js
index d2fd9477..ec12092d 100644
--- a/js/drawingtools/BaseTool.js
+++ b/js/drawingtools/BaseTool.js
@@ -12,7 +12,24 @@
 	
 	ns.BaseTool.prototype.moveToolAt = function(col, row, color, frame, overlay) {};
 
-	ns.BaseTool.prototype.moveUnactiveToolAt = function(col, row, color, frame, overlay) {};
+	ns.BaseTool.prototype.moveUnactiveToolAt = function(col, row, color, frame, overlay) {
+		if (overlay.containsPixel(col, row)) {
+			if (this.previouslyHighlightedPixelCol &&
+				this.previouslyHighlightedPixelRow &&
+				(this.previouslyHighlightedPixelRow != row ||
+					this.previouslyHighlightedPixelCol != col)) {
+
+				// Clean the previously highlighted pixel:
+				overlay.clear();
+			}
+
+			// Show the current pixel targeted by the tool:
+			overlay.setPixel(col, row, Constants.TOOL_TARGET_HIGHLIGHT_COLOR);
+
+			this.previouslyHighlightedPixelCol = col;
+			this.previouslyHighlightedPixelRow = row;	
+		}
+	};
 
 	ns.BaseTool.prototype.releaseToolAt = function(col, row, color, frame, overlay) {};