[Webkit-unassigned] [Bug 8707] event.clientX and event.clientY should be relative to the viewport, not the canvas

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=8707


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #8963|review?                     |review-
               Flag|                            |




------- Comment #10 from darin at apple.com  2006-06-22 09:29 PDT -------
(From update of attachment 8963)
This seems like a big step in the right direction.

But I think it's confusing to have local variables named offsetX and offsetY
and parameter to the MouseEvent/WheelEvent constructors that are entirely
different things than MouseEvent's own offsetX and offsetY properties.

Similarly, it doesn't make sense to me that the clientX and clientY parameters
are no longer clientX and clientY! If their meaning is changing, then the name
should change to. And it should only be called clientX and clientY if that's
really what these are named.

Also, since the WheelEvent constructor does take a pointer to the abstract
view, then we can get from that to the FrameView, so we don't need to add
another parameter to the constructors -- not sure if that's better or not, but
I think it probably is.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.