ooooo.... this is a good find, paul.
I'm going to investigate this.
Erik
On 7/26/07, Paul Spencer <[EMAIL PROTECTED]> wrote:
> 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
>
_______________________________________________
Dev mailing list
[email protected]
http://openlayers.org/mailman/listinfo/dev
