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

[Pg]

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/|
> +-----------------------------------------------------------------+
>
>
>
>
>