Opened 5 years ago

#15272 new bug

Easing demo broken with jQuery 1.12, 2.2, 3.0

Reported by: Joel Davies Owned by:
Priority: minor Milestone: none
Component: ui.effects.* (individual effect) Version: 1.12.1
Keywords: Cc:
Blocked by: Blocking:

Description

The Easing demo doesn't work with jQuery 1.12, 2.2, or 3.0+ (at least in Firefox 58/Chrome 64). Only the default jQuery easing values are shown - none of the additions in jQuery UI are visible. This affects the current documentation on both jqueryui.com and api.jqueryui.com but I have reported here because the problem is the demo itself.

The change was introduced by the new value jQuery.easing._default in jQuery commit 5f2ea40, which returns the string "swing" instead of a function. The problem is that Line 74 in the demo directly executes the value impl, without checking to see if it is a function. At this point, the script crashes both in FF and Chrome.

I can see two ways to fix it. Either check specifically for "_default" and exclude it or check to make sure impl is a function.

jsFiddle examples (these all use jQuery 1.12.1 which is the version currently used for the demo on jqueryui.com)

The only change I made in the latter two is line 11.

Change History (0)

Note: See TracTickets for help on using tickets.