Changeset 216

Timestamp:

07/10/10 21:19:07 (8 weeks ago)

Author:

scott

Message:

Fixed silly and strange bug with event registrations.

• Selections were being added duplicately, but only in IE.
  • The code clearly instructed each selection to be added twice, but for some reason this wasn't happening in Mozilla or Webkit.
  • Removing the addSelection call from the createSelection call, eliminated the redundancy and fixed the bug with duplicate event handlers in IE.

IE based browsers (well, atleast IE 8) are now functioning equivilently to
Webkit-based browsers (well, atleast Chrome) and Mozilla-based browsers (well,
atleast Firefox).

Files:

• xmvc/branches/remove-namespaces/src/main/javascript/xmvc/controller.js (modified) (1 diff)

xmvc/branches/remove-namespaces/src/main/javascript/xmvc/controller.js

@@ -763 +763 @@
         var selection = new Selection(locator)
 
-        this.addSelection(selection)
+/* This is a stupid bug...why was I doing this twice?
+ *      this.addSelection(selection)
+ */
 
         return selection