ng-joyride icon indicating copy to clipboard operation
ng-joyride copied to clipboard

joyride not working with jQuery3

Open schuesslerf opened this issue 8 years ago • 5 comments

I just noticed, that ng-joyride is not working with jQuery3 yet, I tried for hours to get it working until I figured out the error. In ng-joyride.js, line 473 you write "if ($fkEl.size() === 0) {", but size() is deprecated since jQuery 1.8, and it v3 it doesn't seem to work at all any more. If changing .size() to the attribute .length it works fine, but I'm now using jQuery 2.2.4, where it's working without changing anything as well.

Please fix that for the next version (so change .size() to .length), so it works with jQuery3 as well.

schuesslerf avatar Jun 23 '16 14:06 schuesslerf

To confirm, I just noticed this as well -- it looks like the size() function got deprecated in jQuery v1.8, and removed entirely in v3.0 (see this link). It does look like the correct fix would be to change .size() on line 473 to .length.

seanlip avatar Aug 24 '16 09:08 seanlip

Update: I've created a pull request in #68 to fix this.

/cc @abhikmitra

seanlip avatar Aug 30 '16 00:08 seanlip

bump!

ghettosoak avatar Nov 16 '16 14:11 ghettosoak

I created a joyride directive which doesn't rely on jquery or bootstrap, you can check it out here: https://github.com/ahmed-wagdi/angular-joyride

ahmed-wagdi avatar Dec 03 '16 12:12 ahmed-wagdi

+1 on this as well.

Deklin avatar Jul 21 '17 04:07 Deklin