From 27064ff8daf9a8fcdf39cdb0a4d49cd17070fb83 Mon Sep 17 00:00:00 2001
From: Jeff Schiller <codedread@gmail.com>
Date: Tue, 24 Nov 2009 06:54:43 +0000
Subject: [PATCH] Update some TODOs, comment out some no-longer needed code

git-svn-id: http://svg-edit.googlecode.com/svn/trunk@973 eee81c28-f429-11dd-99c0-75d572ba1ddd
---
 editor/svgcanvas.js | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/editor/svgcanvas.js b/editor/svgcanvas.js
index 94e80ee5..06ad2251 100644
--- a/editor/svgcanvas.js
+++ b/editor/svgcanvas.js
@@ -1,10 +1,9 @@
 /*
 	TODOs for TransformList:
 
-	* Ensure rotation works properly (extract the correct rotational center now?)
-		* when groups are resized, center is not found properly (dx/dy are not calculated correctly)
-		* clean up setRotationAngle()
-	* Ensure resized/rotated groups show the proper located and sized selector box
+	* Fix Opera's centering of rotated, resized groups
+	* Fix resizing of rotated already-resized groups (scales incorrect with mouse)
+	* Ensure groups with rotated children have properly located and sized selector box
 	* Ensure ungrouping works (Issue 204)
 */
 /*
@@ -2524,7 +2523,7 @@ function BatchCommand(text) {
 					dx = r * Math.cos(theta);
 					dy = r * Math.sin(theta);
 				}
-				
+
 				// if not stretching in y direction, set dy to 0
 				// if not stretching in x direction, set dx to 0
 				if(current_resize_mode.indexOf("n")==-1 && current_resize_mode.indexOf("s")==-1) {
@@ -2558,7 +2557,7 @@ function BatchCommand(text) {
 				translateOrigin.setTranslate(-(left+tx),-(top+ty));
 				if(evt.shiftKey) {
 					if(sx == 1) sx = sy
-						else sy = sx;
+					else sy = sx;
 				}
 				scale.setScale(sx,sy);
 
@@ -2582,7 +2581,9 @@ function BatchCommand(text) {
 				if (ty) {
 					selectedBBox.y += dy;
 				}
-				
+
+				// I believe the following commented-out code is no longer required - schiller				
+				/*
 				// update box width/height
 				selectedBBox.width = round(width*sx);
 				selectedBBox.height = round(height*sy);
@@ -2608,8 +2609,9 @@ function BatchCommand(text) {
 						selectedBBox.y -= selectedBBox.height;
 					}
 				}
+				*/
 				
-				selectorManager.requestSelector(selected).resize();//selectedBBox); // TODO: remove box arg
+				selectorManager.requestSelector(selected).resize();
 				break;
 			case "zoom":
 				x *= current_zoom;