Hey devs,

something else just came up.  As part of removing prototype.js  
dependency, it appears that you have kept most of Prototype's event  
handling code, including the following (lines 327-331 of Events.js):

if (window.Event) {
   OpenLayers.Util.extend(window.Event, OpenLayers.Event);
} else {
   var Event = OpenLayers.Event;
}

If you include OpenLayers.js AFTER including prototype.js, this  
effectively replaces most of prototype's Event object with the  
OpenLayers.Event object, which has undesirable results (i.e. calling  
Event.observe is actually calling OpenLayers.Event.observe)

Is there a particular reason why we are putting OpenLayers.Event into  
the global namespace?

Paul


+-----------------------------------------------------------------+
|Paul Spencer                          [EMAIL PROTECTED]    |
+-----------------------------------------------------------------+
|Chief Technology Officer                                         |
|DM Solutions Group Inc                http://www.dmsolutions.ca/ |
+-----------------------------------------------------------------+





_______________________________________________
Dev mailing list
[email protected]
http://openlayers.org/mailman/listinfo/dev

Reply via email to