You're correct it's not yet inside CVS. I wan'ted to
submit a few other fixes as well. This fix should make
it's way into cvs over the weekend.

--
Raymond Irving

--- "SourceForge.net" <[EMAIL PROTECTED]> wrote:
> Bugs item #762495, was opened at 2003-06-28 22:31
> Message generated for change (Comment added) made by
> bs_php
> You can respond by visiting: 
>
https://sourceforge.net/tracker/?func=detail&atid=105757&aid=762495&group_id=5757
> 
> Category: DynAPI 3 API
> Group: Unverified
> Status: Open
> Resolution: Fixed
> Priority: 5
> Submitted By: Sam Blum (bs_php)
> Assigned to: Nobody/Anonymous (nobody)
> Summary: Wrong position retured when page has been
> scrolled down.
> 
> Initial Comment:
> Reproduced the bug in IE and Mozilla 1.3
> 
> To see the effect, open the dynapi example:
> examples/dynapi.api.mouseevent.html 
> - You should see some colored layers that are
> cascaded 
> in each 
>   other in the middle of the page. 
> - On the left we have some links to turn
> mouse-events 
> on and off.
> - The debugger window should open as well.
> 
> The page contains so much info, that a standard
> scroll-
> bar should appear on the right of the browser. If
> not 
> make the browser window less high.
> 
> Now do as follows:
> A) Turn on the click/dblclick events for the blue
> layer. 
>    This is done by clicking on the 
>       "Listen to click/dblclick events -> blue ->
> on" 
>    link (see the page)
> 
> B) Then click on the blue layer top left corner and
> look 
>    at the output on the debugger window. It should
> say 
>    something like this: 
>    "click blue xy=0,0 pXY=300,200 origin=blue"
> 
>    NOTE: 
>    xy are the coordinates of the click on the blue
> layer 
>    relative to it's own top left corner. pXY are the
> 
>    absolute coordinates relative to the view (top
> left 
>    corner of the browser).
> 
> C) Now scroll down a little and click again on the
> blue 
>    layer (try to hit the same spot). Now in the
> debugger 
>    window look at the result: 
>    "click blue xy=0,-148 pXY=300,52 origin=blue"
> 
> BUG: 
>   Compare the xy result 0,0 and 0,-148. I would have
> 
>   expected to get the same result in case A) and B) 
>   for xy (the pXY results are OK). But the y axis
> has 
>   a negative number, caused by the scrolling.
>   
>   
> Side Effect Bug:  
>   Next I modified the code and set the red layer 
> to 'relative'
>   Thus, I added line:  red.setPosition('relative');
>   This will position the layers at the bottom of the
> 
>   page. Again try to repeat B) and C) (as good as
> you 
> can) 
>   and compare the results:
>     "click blue xy=8,496 pXY=308,696 origin=blue"
>     "click blue xy=8,396 pXY=308,596 origin=blue"
> 
> The effect is the same AND a *side effect* shows up:
> alway have an x offset of 8px (13px on IE). But I
> heard 
> that reletive posistioning is problematic.
> 
> -- bs_php
> 
>
----------------------------------------------------------------------
> 
> >Comment By: Sam Blum (bs_php)
> Date: 2003-07-04 00:44
> 
> Message:
> Logged In: YES 
> user_id=308859
> 
> According to the CVS on sourceforge, no updates have
> been 
> mad to mouse_dom.js in dynapi3x.
> 
> Are you using an other CVS? If so can I get access?
> 
> -- sam
> 
>
----------------------------------------------------------------------
> 
> Comment By: Raymond Irving (xwisdom)
> Date: 2003-06-30 14:55
> 
> Message:
> Logged In: YES 
> user_id=696242
> 
> Hi bs_php,
> 
> Bug verified and fixed
> 
> I've implemented your fix into the MouseEvent files
> and it 
> seems to have resolve the porblem. Many thanks for
> this fix.
> 
> --
> Raymond Irving
> 
>
----------------------------------------------------------------------
> 
> Comment By: Sam Blum (bs_php)
> Date: 2003-06-29 01:18
> 
> Message:
> Logged In: YES 
> user_id=308859
> 
> This would my suggestion for mouse_dom.js
> 
> p.getScrollX = function() {return 
> document.body.scrollTop||0};
> p.getScrollY = function() {return 
> document.body.scrollLeft||0};
> p.getX=function() {return (this.x||0)  +
> this.getScrollX()};
> p.getY=function() {return (this.y||0)  +
> this.getScrollY()};
> 
> 
> This would my suggestion for mouse_ie.js
> p.getScrollX = function() {return
> window.pageXOffset||0};
> p.getScrollY = function() {return
> window.pageYOffset||0};
> p.getX=function() {return (this.x||0)  +
> this.getScrollX()};
> p.getY=function() {return (this.y||0)  +
> this.getScrollY()};
> 
> 
> 
>
----------------------------------------------------------------------
> 
> Comment By: Sam Blum (bs_php)
> Date: 2003-06-29 00:47
> 
> Message:
> Logged In: YES 
> user_id=308859
> 
> OK here is the solution ( I know it' currently a
> hack) 
> =================================
> For Mozzila 1.3 in mouse_dom.js replace the lines:
> 
>   p.getX = function() {return this.x};
>   p.getY = function() {return this.y};
> 
> with 
> 
>   p.getX_ = function() {
>     var _x = this.x||0;
>     var _xOffset = 0;
>     if (this.ns) {
>       _xOffset = document.body.scrollTop||0;
>     }else{
>       _xOffset = window.pageXOffset||0;
>     }
>     return _x + _xOffset;
>   };
>   
>   p.getY_ = function() {
>     var _y = this.y||0;
>     var _yOffset = 0;
>     if (this.ns) {
> 
=== message truncated ===


__________________________________
Do you Yahoo!?
SBC Yahoo! DSL - Now only $29.95 per month!
http://sbc.yahoo.com


-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://www.mail-archive.com/[EMAIL PROTECTED]/

Reply via email to