#7318 closed feature (fixed)

Opened May 04, 2011 03:18AM UTC

Closed June 21, 2011 06:18AM UTC

Add the queue functions to $.fn.effect()

Reported by: gnarf Owned by: gnarf
Priority: major Milestone: 1.9.0
Component: ui.effects.core Version: 1.8.12
Keywords: Cc:
Blocked by: Blocking:
Description

Every effect in effects.effect returns this.queue(). We could DRY out the effects a bit while we are upgrading to 1.9 if we were to handle the "queue" issues in the main effects function.

Attachments (0)
Change History (4)

Changed May 04, 2011 03:19AM UTC by gnarf comment:1

description: '''Every''' effect in effects.effect returns this.queue(). We could DRY out the API a bit while we are upgrading to 1.9. → '''Every''' effect in effects.effect returns this.queue(). We could DRY out the effects a bit while we are upgrading to 1.9 if we were to handle the "queue" issues in the main effects function.
owner: → gnarf
priority: minor → major
status: new → assigned

Changed May 04, 2011 12:49PM UTC by scottgonzalez comment:2

type: bug → feature

Changed June 21, 2011 06:10AM UTC by gnarf comment:3

Further amending this... All of the effects also are handing calling the complete callback - after putting the queue stuff into 

$.fn.effect
its now possible to wrap the complete function up with it and just pass one 
 done
argument for the effect to call when it finishes.

Changed June 21, 2011 06:18AM UTC by gnarf comment:4

resolution: → fixed
status: assigned → closed

Effects.*: Updating Effect Method API to avoid duplicating the queue call - Fixes #7318 - Add the queue functions to $.fn.effect()

Changeset: 1c1a3b1a361d90a73755fbd038b3cdfb0960c29f