Remove jQuery Update dependency [#1862494]

Comment	File	Size	Author
#5	views_isotope-nojqueryupdate-1862494-5.patch	2.6 KB	sherakama
#4	views_isotope-remove-dependency-1862494-4.patch	883 bytes	Chris Gillis

Comment #1

Daniel_Rempe CreditAttribution: Daniel_Rempe commented 13 May 2013 at 13:11

I tested the same (drupal 7.22 and views_isotope without jquery_update) because jquery_update makes it very difficult to work with the media module. So far I couldn't find anything not working. So it seems to be safe to remove this dependency.

Log in or register to post comments

Comment #2

jennifer.beatty CreditAttribution: jennifer.beatty commented 29 October 2013 at 17:01

I too was having issues with jQuery Update (javascript in the front and back end of my site was disabled- even when I selected v1.5). I removed the dependency of Views Isotope on jQuery Update and everything is working well. Hopefully I'll be able to move forward successfully without jQuery Update enabled.

Log in or register to post comments

Comment #3

kreynen CreditAttribution: kreynen commented 4 November 2013 at 12:28

Issue summary:	View changes
Status:	Active	» Needs work

JQuery Update is no longer required and will be dropped as a dependency in the next release.

Log in or register to post comments

Comment #4

Chris Gillis CreditAttribution: Chris Gillis commented 4 June 2014 at 08:28

Title:	Is jQuery Update dependency really necessary?	» Remove jQuery Update dependency
Version:	7.x-1.x-dev	» 7.x-2.x-dev
Category:	Support request	» Task
Status:	Needs work	» Reviewed & tested by the community

File	Size
views_isotope-remove-dependency-1862494-4.patch	883 bytes

Patch attached... Can I be so bold as to mark it tested?

Log in or register to post comments

Comment #5

sherakama CreditAttribution: sherakama commented 25 February 2015 at 00:42

File	Size
views_isotope-nojqueryupdate-1862494-5.patch	2.6 KB

Can I be so bold as to ask for adding this patch to the 1.x branch? I have a use case where I do not want to add jquery_update and all of its baggage to my project.

Included in patch
1. Removal of jquery_update requirement
2. Addition of runtime requirement suggestion
3. Make file that downloads and places the jquery.isotope and jquery.imagesloaded plugins

Thanks,

Log in or register to post comments

Comment #6

Chris Gillis CreditAttribution: Chris Gillis commented 11 October 2015 at 10:15

Fixed in 7.x-2.0-beta1.

If people are willing to test #5 against the 1.x branch I can commit it.

Log in or register to post comments

Comment #7

Chris Gillis CreditAttribution: Chris Gillis commented 11 October 2015 at 10:15

Version:

7.x-2.x-dev

» 7.x-1.x-dev

Log in or register to post comments

Comment #8

Chris Gillis CreditAttribution: Chris Gillis commented 11 October 2015 at 13:36

Status:

Reviewed & tested by the community

» Needs review

Log in or register to post comments

Comment #9

Chris Gillis CreditAttribution: Chris Gillis commented 22 January 2016 at 18:41

@sherakama, let me know if this is still a requirement for you, otherwise I might go ahead and close this ticket. I personally don't have time to test patches against old beta versions, and I recommend you just apply the patch yourself, until you are ready to upgrade.