Opened 13 years ago
Closed 13 years ago
#2032 closed bug (fixed)
clientLeft should be clientTop in event.fix?
| Reported by: | arrix | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 1.2.2 |
| Component: | event | Version: | 1.2.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
event.pageY = event.clientY + (doc && doc.scrollTop || body && body.scrollTop || 0) - (doc.clientLeft || 0);
I think doc.clientLeft should be doc.clientTop
Change History (2)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
It was a mistake and is now fixed in Rev [4099]
Note: See
TracTickets for help on using
tickets.
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.