[ka-Map-users] kaTool and scrollbar: patch of the patch

[Paul Spencer]

Pg, committed to cvs.

Thanks,

Paul

On 4-May-06, at 3:46 PM, Pg wrote:

> Thanks, but there is still a fix
>
> kaTool.js, line 500, you write
>
>    /*
>     * should this be used here instead of adjustPixPosition?
>    var x = e.pageX || (e.clientX +
>        (document.documentElement.scrollLeft ||  
> document.body.scrollLeft));
>    var y = e.pageY || (e.clientY +
>        (document.documentElement.scrollTop ||  
> document.body.scrollTop));
>    */
>    var a = this.adjustPixPosition( e.clientX, e.clientY );
>
>
> The answer is yes, you shuld change this in:
>
>    var x = e.pageX || (e.clientX +
>        (document.documentElement.scrollLeft ||  
> document.body.scrollLeft));
>    var y = e.pageY || (e.clientY +
>        (document.documentElement.scrollTop ||  
> document.body.scrollTop));
>    var a = this.adjustPixPosition( x, y );
>
>
> Bye
>
>
> 2006/5/3, Paul Spencer <pspencer at dmsolutions.ca>:
>> Pg,
>>
>> I've applied this to cvs.
>>
>> Cheers
>>
>> Paul
>>
>> On 3-May-06, at 11:29 AM, Pg wrote:
>>
>> > There is a small error in kaTool CVS revision 1.28.
>> >
>> > This is the patch:
>> >
>> > Index: kaTool.js
>> > ===================================================================
>> > --- kaTool.js (revision 60)
>> > +++ kaTool.js (working copy)
>> > @@ -414,8 +414,8 @@
>> >
>> >     this.kaMap.checkWrap.apply(this.kaMap, []);
>> >
>> > -    this.lastx=e.clientX;
>> > -    this.lasty=e.clientY;
>> > +    this.lastx=x;
>> > +    this.lasty=y;
>> >     return false;
>> > };
>> >
>> > _______________________________________________
>> > ka-Map-users mailing list
>> > ka-Map-users at lists.maptools.org
>> > http://lists.maptools.org/mailman/listinfo/ka-map-users
>>
>> +-----------------------------------------------------------------+
>> |Paul Spencer                           pspencer at dmsolutions.ca   |
>> +-----------------------------------------------------------------+
>> |Applications & Software Development                              |
>> |DM Solutions Group Inc                 http://www.dmsolutions.ca/|
>> +-----------------------------------------------------------------+
>>
>>
>>
>>
>>

+-----------------------------------------------------------------+
|Paul Spencer                           pspencer at dmsolutions.ca   |
+-----------------------------------------------------------------+
|Applications & Software Development                              |
|DM Solutions Group Inc                 http://www.dmsolutions.ca/|
+-----------------------------------------------------------------+