Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#56702 closed defect (bug) (fixed)

Twenty Seventeen: Update jQuery scrollTo to the latest version

Reported by: desrosj's profile desrosj Owned by: sarahricker's profile sarahricker
Milestone: 6.1 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: good-first-bug has-patch commit
Focuses: Cc:

Description

The scrollTo jQuery plugin is included in Twenty Seventeen, currently at version 2.1.2. Version 2.1.3 is now available, so this should be updated.

In 2.1.3, jQuery.isFunction is removed since it has been deprecated.

A full list of changes can be found on GitHub.

Change History (11)

#1 @desrosj
2 years ago

  • Keywords good-first-bug added

This ticket was mentioned in PR #3381 on WordPress/wordpress-develop by sarahricker.


2 years ago
#2

  • Keywords has-patch added; needs-patch removed

Trac ticket: https://core.trac.wordpress.org/ticket/56702
Updating scrollTo jQuery plugin included in Twenty Seventeen, from version 2.1.2 to version 2.1.3.

Testing Note:
In 2.1.3, jQuery.isFunction is removed since it has been deprecated.
A full list of changes can be found on GitHub.

#3 @sarahricker
2 years ago

  • Keywords needs-testing added
  • Owner set to sarahricker
  • Status changed from new to accepted

#4 @desrosj
2 years ago

  • Keywords changes-requested added; needs-testing removed

Added review notes on the PR.

sarahricker commented on PR #3381:


2 years ago
#5

Thanks @desrosj! The reason for the excessive changes is actually that we don't seem to have a true copy of 2.1.2. If you checkout before commit 18fb646 and compare our version to the file in jquery.scrollTo/releases/tag/2.1.2 and run a diff, there is quite a difference. ☹️

As such, we should probably do some extra testing to be sure other stuff isn't breaking. But, it's probably good to get the true version in there - what do you think?

I'll work on updating the version specified in the wp_enqueue_script() call in functions.php in the meantime. Thanks for the catch!

#6 @sarahricker
2 years ago

  • Keywords needs-testing added; changes-requested removed

#7 @mukesh27
2 years ago

Thanks @sarahricker

PR #3381 looks good. Need final review from @desrosj

#8 @desrosj
2 years ago

  • Keywords commit added; needs-testing removed

#9 @desrosj
2 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 54374:

Twenty Seventeen: Update the scrollTo jQuery plugin.

This updates the scrollTo jQuery plugin included in Twenty Seventeen to the latest version, 2.1.3.

For a full list of changes in this update, see GitHub: https://github.com/flesler/jquery.scrollTo/compare/2.1.2...v2.1.3.

Props sarahricker, mukesh27, desrosj.
Fixes #56702.

sarahricker commented on PR #3381:


2 years ago
#11

Awesome, thanks!

Note: See TracTickets for help on using tickets.