From ae474131cc5ab75dbb0e200ff4cc03ec37f771a2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BCrg=20Lehni?= <juerg@scratchdisk.com>
Date: Thu, 13 Jun 2013 10:40:54 -0700
Subject: [PATCH] Implement stroke hittesting for Shape.Rectangle.

---
 src/item/Shape.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/item/Shape.js b/src/item/Shape.js
index d6252e96..8f633103 100644
--- a/src/item/Shape.js
+++ b/src/item/Shape.js
@@ -69,7 +69,7 @@ var Shape = Item.extend(/** @lends Shape# */{
 
 	_getBounds: function(getter, matrix) {
 		var rect = new Rectangle(this._size).setCenter(0, 0);
-		if (this.hasStroke())
+		if (getter !== 'getBounds' && this.hasStroke())
 			rect = rect.expand(this.getStrokeWidth());
 		return matrix ? matrix._transformBounds(rect) : rect;
 	},
@@ -90,7 +90,11 @@ var Shape = Item.extend(/** @lends Shape# */{
 				strokeWidth = this.getStrokeWidth();
 			switch (type) {
 			case 'rect':
-				// TODO: Implement stroke!
+				var rect = new Rectangle(this._size).setCenter(0, 0),
+					outer = rect.expand(strokeWidth),
+					inner = rect.expand(-strokeWidth);
+				if (outer._containsPoint(point) && !inner._containsPoint(point))
+					return new HitResult('stroke', this);
 				break;
 			case 'circle':
 			case 'ellipse':
@@ -110,6 +114,7 @@ var Shape = Item.extend(/** @lends Shape# */{
 				}
 				if (2 * Math.abs(point.getLength() - radius) <= strokeWidth)
 					return new HitResult('stroke', this);
+				break;
 			}
 		}
 		return _hitTest.base.apply(this, arguments);