Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#5083 closed bug (fixed)

class animation functions need to accept function setters

Reported by: Scott González Owned by:
Priority: blocker Milestone: 1.8
Component: ui.effects.core Version: 1.7.2
Keywords: Cc:
Blocked by: Blocking:

Description

.addClass(), .removeClass(), .toggleClass() all allow passing a function as a setter in jQuery 1.4. The overrides for class animations need to be updated to match the core API.

Change History (2)

comment:1 Changed 10 years ago by Jörn Zaefferer

Resolution: fixed
Status: newclosed

Works fine, as far as I can tell. See http://jquery-ui.googlecode.com/svn/trunk/tests/visual/effects.all.html Check Firebug console for the correct arguments, its all there. Tested in FF3.5 and IE6.

comment:2 Changed 10 years ago by Jörn Zaefferer

To calculate the styles to animate, the normale add/remove/toggle method is called, which therefore delegates the parameter-evaluation back to jQuery. Therefore this just works.

Note: See TracTickets for help on using tickets.