Opened 5 years ago

Last modified 5 years ago

#10539 open feature

Effects: Enhanced effect methods should support the full API from core

Reported by: sJoris Owned by:
Priority: minor Milestone: none
Component: ui.effects.core Version: 1.11.0
Keywords: Cc:
Blocked by: Blocking:

Description

The jQuery UI functions .show(), .hide() and .toggle() should extend the jQuery core functions, but all the functionality of the Core functions break when using the options argument with the extended property effect.

The properties of the Core functions that stop working are: start, step, progress, done, fail and always.

It seems it is not possible to 'mix' properties of jQuery UI with properties of the jQuery Core, but that is what is expected when it says "This plugin extends jQuery's built-in method".

Test case: http://jsfiddle.net/eoqrkL88/

The Unit test on line 21 (.hide() with step) also fails if the property effect is added.

Related: Ticket #9126.

Change History (2)

comment:1 Changed 5 years ago by Scott González

Status: newopen
Summary: Effects: show/hide/toggle Core functions break when adding property EffectEffects: Enhanced effect methods should support the full API from core
Type: bugfeature

"This plugin extends jQuery's built-in method" means that the existing APIs still exist and new APIs have been added. It doesn't say anything about the functionality of the new APIs, other than that they don't break the existing APIs.

comment:2 Changed 5 years ago by mikesherov

Just as a note, they also don't support "step".

Note: See TracTickets for help on using tickets.