jQuery: The Write Less, Do More JavaScript Library

Ticket #2032 (closed bug: fixed)

Opened 5 months ago

Last modified 5 months ago

clientLeft should be clientTop in event.fix?

Reported by: arrix Assigned to: anonymous
Type: bug Priority: major
Milestone: 1.2.2 Component: event
Version: 1.2.1 Keywords:
Cc: Needs: Review

Description 

event.pageY = event.clientY + (doc && doc.scrollTop || body && body.scrollTop || 0) - (doc.clientLeft || 0);

I think doc.clientLeft should be doc.clientTop

Attachments

Change History

Changed 5 months ago by davidserduke

This issue is not in 1.2.1 but rather in SVN rev 4092 (and earlier). It also appears to be in Brandon's event enhancement. I agree with the author this looks odd. If it is correct we should probably put in a meaningful comment to explaining what is going on.

Changed 5 months ago by brandon

  • status changed from new to closed
  • resolution set to fixed

It was a mistake and is now fixed in Rev [4099]

Note: See TracTickets for help on using tickets.