Bug Tracker

Opened 9 years ago

Closed 9 years ago

#6976 closed bug

.offset(fn) misbehaves if fn changes its argument

Reported by: dmethvin Owned by: dmethvin
Priority: low Milestone: 1.4.3
Component: offset Version: 1.4.2
Keywords: offset returns Cc:
Blocked by: Blocking:

Description

We should either document the current behavior or change it; are there other .setter(fn) methods that have the same issue?

See this thread:

http://forum.jquery.com/topic/should-position-make-a-defensive-copy-of-the-position-object-before-passing-it-to-a-callback

Change History (2)

comment:1 Changed 9 years ago by addyosmani

Keywords: offset returns added
Owner: set to dmethvin
Priority: low
Status: newpending

Could you supply us with a test case (using your modification) so that we can accurately test, verify and establish whether this fix would work correctly and also whether it breaks any other functionality in the process?

comment:2 Changed 9 years ago by trac-o-bot

Status: pendingclosed

Automatically closed due to 14 days of inactivity.

Note: See TracTickets for help on using tickets.