Bug Tracker

Modify ↓

Ticket #6976 (closed bug)

Opened 3 years ago

Last modified 3 years ago

.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:
Blocking: Blocked by:

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

comment:1 Changed 3 years ago by addyosmani

  • Keywords offset returns added
  • Owner set to dmethvin
  • Status changed from new to pending
  • Priority set to low

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 3 years ago by trac-o-bot

  • Status changed from pending to closed

Automatically closed due to 14 days of inactivity.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.