[OpenLayers-Dev] Ticket #1359: spelling mistake?

Erik Uzureau euzuro at gmail.com
Thu Oct 23 18:04:23 EDT 2008


good find, chris. indeed a bug. i've reopened the ticket and am preparing a
patch:
http://trac.openlayers.org/ticket/1359#comment:12

On Thu, Oct 23, 2008 at 4:47 PM, Chris Rankin <rankincj at yahoo.com> wrote:

> Hi,
>
> I have noticed the following oddity in a patch attached to ticket #1359, at
> line 152:
>
> http://trac.openlayers.org/attachment/ticket/1359/rightClicks.patch
>
>        144             var propagate = true;
>        145
>        146             // Collect right mouse clicks from the mouseup
>        147             //  IE - ignores the second right click in mousedown
> so using
>        148             //  mouseup instead
>        149             if (this.checkModifiers(evt) &&
>        150                 this.control.handleRightClicks &&
>        151                 OpenLayers.Event.isRightClick(evt)) {
>        152               propogate = this.rightclick(evt);
>        153             }
>        154
>        155             return propagate;
>
> Surely that should be "propagate" rather than "propogate"?
>
> Cheers,
> Chris
>
> P.S. Is there any reason why the following would NOT notify me of
> right-clicks on my map?
>
>            map.events.register('rightclick', map, function(e) {
>                alert(this.CLASS_NAME);
>                return true;
>            });
>
> I have already set the handleRightClicks property to "true" on the
> Navigation control.
>
>
>
> _______________________________________________
> Dev mailing list
> Dev at openlayers.org
> http://openlayers.org/mailman/listinfo/dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.osgeo.org/pipermail/openlayers-dev/attachments/20081023/bfec36d3/attachment.html


More information about the Dev mailing list