Twenty Sixteen: Deprecated jQuery.fn.unbind

[Malae]

I posted this 2 years ago, but it has not been fixed in Theme: Twenty Sixteen
jQuery.fn.unbind() is deprecated in /twentysixteen/js/functions.js Line 97
siteNavigation.find( '.menu-item-has-children > a' ).unbind( 'touchstart.twentysixteen' );
should be changed to:
siteNavigation.find( '.menu-item-has-children > a' ).on("unbind", 'touchstart.twentysixteen' );

Trac ticket: https://core.trac.wordpress.org/ticket/58225

[dilipbheda]

@Malae I think jQuery Off method will handle to remove the event I think the jQuery Off method will help to remove the registered event instead of on("unbind", 'touchstart.twentysixteen' );

cc @audrasjb

Thanks

[sabernhardt]

Also: the .unbind() method is found in Twenty Fourteen's slider.js and Twenty Seventeen's navigation.js.

[audrasjb]

Works fine in my testing. Marking for commit.

@dilipbheda @sabernhardt feel free to open other tickets to handle this for T14 and T17 :)

[mukesh27]

unbind used many places in core. Why it give error here?

[audrasjb]

In 56102:

Twenty Sixteen: Replace deprecated unbind method with off.

This fixes a Deprecated jQuery.fn.unbind() notice.

Props Malae, dilipbheda, sabernhardt, audrasjb.
Fixes #58225.

committed in https://core.trac.wordpress.org/changeset/56102