Revert "annotorious#193 added removeHandler method"

readme.md

@@ -1,10 +1,3 @@
-To Do:
-1. removeHandler https://github.com/annotorious/annotorious/issues/193
-2. displacement of annotation in chrome
-3. circular annotation - https://github.com/annotorious/annotorious/issues/55
-4. Drawing a polygon using annotorious - update this in gitter openseadragon
-6. Adding custom annotations on OpenSeadragon module
-
 # Annotorious - Image Annotation for the Web
 
 **CURRENTLY UNSUPPORTED**

src/annotorious.js

@@ -126,17 +126,6 @@ annotorious.Annotorious.prototype.addHandler = function(type, handler) {
   });
 }
 
-/**
- * Removes an event handler to Annotorious.
- * @param {annotorious.events.EventType} type the event type
- * @param {Function} handler the handler function
- */
-annotorious.Annotorious.prototype.removeHandler = function(type, handler) {
-  goog.array.forEach(this._modules, function(module) {
-    module.removeHandler(type, handler);
-  });
-}
-
 /**
  * Adds a plugin to Annotorious.
  * @param {string} plugin_name the plugin name

src/mediatypes/annotator.js

@@ -14,10 +14,6 @@ annotorious.mediatypes.Annotator.prototype.addHandler = function(type, handler)
   this._eventBroker.addHandler(type, handler);  
 }
 
-annotorious.mediatypes.Annotator.prototype.removeHandler = function(type) {
-  this._eventBroker.removeHandler(type);  
-}
-
 annotorious.mediatypes.Annotator.prototype.fireEvent = function(type, event, opt_extra) {
   return this._eventBroker.fireEvent(type, event, opt_extra);
 }

src/mediatypes/module.js

@@ -310,24 +310,6 @@ annotorious.mediatypes.Module.prototype.addHandler = function(type, handler) {
   this._eventHandlers.push({ type: type, handler: handler });
 }
 
-/**
- * Removes a/all lifecycle event handler/s to this module.
- * @param {annotorious.events.EventType} type the event type
- * @param {Function} handler the handler function (optional)
- */
-annotorious.mediatypes.Module.prototype.removeHandler = function(type, handler) {
-  goog.array.forEach(this._annotators.getValues(), function(annotator, idx, array) {
-    annotator.removeHandler(type, handler);
-  });
-    goog.array.forEach(this._eventHandlers, function(elem, index, array) {
-        if(elem.type === type) {
-            if(!handler || elem.handler === handler) {
-                goog.array.removeAt(array, index);
-            }         
-        }
-    });
-}
-
 /**
  * Adds a plugin to this module.
  * @param {Plugin} plugin the plugin

src/mediatypes/openseadragon/openseadragon.annotator.js

@@ -149,7 +149,9 @@ annotorious.mediatypes.openseadragon.OpenSeadragonAnnotator.prototype.editAnnota
 }
 
 annotorious.mediatypes.openseadragon.OpenSeadragonAnnotator.prototype.fromItemCoordinates = function(itemCoords) {
-  var offset = annotorious.dom.getOffset(this.element);
+  var offset = annotorious.dom.getOffset(this.element); 
+  offset.top += window.pageYOffset;
+  offset.left += window.pageXOffset;
 
   var viewportPoint = new OpenSeadragon.Point(itemCoords.x, itemCoords.y);
   var viewportOpposite = (itemCoords.width) ? new OpenSeadragon.Point(itemCoords.x + itemCoords.width, itemCoords.y + itemCoords.height) : false; 
@@ -191,7 +193,9 @@ annotorious.mediatypes.openseadragon.OpenSeadragonAnnotator.prototype.getActiveS
 
 
 annotorious.mediatypes.openseadragon.OpenSeadragonAnnotator.prototype.toItemCoordinates = function(xy) {
-  var offset = annotorious.dom.getOffset(this.element);
+  var offset = annotorious.dom.getOffset(this.element); 
+  offset.top += window.pageYOffset;
+  offset.left += window.pageXOffset;
 
   var viewportPoint = new OpenSeadragon.Point(xy.x + offset.left, xy.y + offset.top);
   var viewportOpposite = (xy.width) ? new OpenSeadragon.Point(xy.x + offset.left + xy.width - 2, xy.y + offset.top + xy.height - 2) : false;