Util.getPosition() fix for mouse pointer offset when page is scrolled down

[erikgull]

This commit partly fixes #258 for the case when the canvas is not scaled.

All credits to Brian Huisman, the author of http://www.greywyvern.com/?post=331 .

[kanaka]

WIth a quick search I was not able to determine the license of this code. Please update if you can provide information on the license. Thanks.

[erikgull]

I contacted Brian Huisman, and he states the following in the e-mail reply: "Unless otherwise noted, code on my site is public domain. You may credit me if you like, but it's not necessary.".

[emmar]

This fix also needs to be applied to scrollLeft. You don't have to include my code as your fix, but this is how I changed erikgull's scrollDist function to make the mouse position correct when you scroll either horizontally or vertically.

function scrollDist() 
        {
        var myScrollTop, myScrollLeft;
        var html = document.getElementsByTagName('html')[0];

        // get the scrollTop part
        if (html.scrollTop && document.documentElement.scrollTop) 
            {myScrollTop = html.scrollTop;} 
        else if (html.scrollTop || document.documentElement.scrollTop) 
            {myScrollTop = html.scrollTop + document.documentElement.scrollTop;} 
        else if (document.body.scrollTop) 
            {myScrollTop = document.body.scrollTop;}
        else {myScrollTop = 0;}

        // get the scrollLeft part
        if (html.scrollLeft && document.documentElement.scrollLeft) 
            {myScrollLeft = html.scrollLeft;} 
        else if (html.scrollLeft || document.documentElement.scrollLeft) 
            {myScrollLeft = html.scrollLeft + document.documentElement.scrollLeft;} 
        else if (document.body.scrollLeft) 
            {myScrollLeft = document.body.scrollLeft;}
        else {myScrollLeft = 0;}

        return [myScrollLeft, myScrollTop];
        };

[denouche]

Will this patch be merged ?
I really need it !
Thank you for your help.

[samhed]

The problem which @emmar mentioned would need to be included as well. @erikgull can you fix this?

[DirectXMan12]

I'm going to go ahead an fix this.