[OpenLayers-Dev] Ticket #1359: spelling mistake?
Erik Uzureau
euzuro at gmail.com
Thu Oct 23 18:16:08 EDT 2008
patch added, ticket up for review
http://trac.openlayers.org/attachment/ticket/1359/propagate.patch
can you let us know if that fixes your problem?
erik
On Thu, Oct 23, 2008 at 5:04 PM, Erik Uzureau <euzuro at gmail.com> wrote:
> 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/d50ebd25/attachment.html
More information about the Dev
mailing list