#9126 closed bug (fixed)

Opened February 27, 2013 02:58AM UTC

Closed February 28, 2013 06:35PM UTC

Last modified August 11, 2014 12:09PM UTC

.show()/.hide() do not support all of core's options

Reported by: albi Owned by:
Priority: minor Milestone: 1.10.2
Component: ui.effects.core Version: 1.10.1
Keywords: Cc:
Blocked by: Blocking:
Description

.hide() and .show() are refined in jquery ui not just extended !! this cause them to break some of their functionalities like STEP function;

http://jsfiddle.net/L5JV7/1/

Attachments (0)
Change History (5)

Changed February 27, 2013 03:30AM UTC by tj.vantoll comment:1

component: ui.effects.* (individual effect) → ui.effects.core
status: new → open
summary: Jquery UI breaks jQuery .hide and .show() → .show()/.hide() do not support all of core's options

Changed February 28, 2013 06:35PM UTC by Scott González comment:2

resolution: → fixed
status: open → closed

Effects: Handle the .hide/show/toggle( options ) signatures from core properly. Fixes #9126 - .show()/.hide() do not support all of core's options.

Changeset: 6f2957743659387f66c9b8953bba9fac5617a440

Changed February 28, 2013 06:35PM UTC by scottgonzalez comment:3

milestone: none → 1.10.2

Changed August 11, 2014 09:12AM UTC by sJoris comment:4

This is still a bug in the latest jQuery UI 1.11.0 if you use the property effect to change the show/hide-effect. Then the 'step' and 'progress' functions stop working.

See: http://jsfiddle.net/eoqrkL88/

So the Unit test https://github.com/jquery/jquery-ui/blob/master/tests/unit/effects/effects_core.js on line 21 (''.hide() with step'') should also be changed or some extra tests should be added.

Changed August 11, 2014 12:09PM UTC by scottgonzalez comment:5

_comment0: Please file a separate issue as this is about adding feature parity to jQuery UI easings, not about broken features from jQuery Core. → 1407759065045352

Please file a separate issue as this is about adding feature parity to jQuery UI effects, not about broken features from jQuery Core.