Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove jquery script registration from juice library #36

Open
rustd opened this issue Feb 25, 2013 · 1 comment
Open

Remove jquery script registration from juice library #36

rustd opened this issue Feb 25, 2013 · 1 comment

Comments

@rustd
Copy link

rustd commented Feb 25, 2013

Today as it is, juice registers jquery in
https://github.com/appendto/juiceui/blob/master/Juice/JuiceApp.cs

In VS2012, all the webforms templates demonstrate registering jquery through AspNet.ScriptManager.jQuery package. This package makes it easier to register and update jquery scripts.
If you install juice in such projects then the user experience is broken since juice updates the jquery reference to a fixed version which is not present in the project

Juice library should be updated to remove the jquery dependancy from code and take a dependancy on this package

@shellscape
Copy link
Contributor

(just adding the comment for posterity) While I'm not an active maintainer of Juice any longer, I'm not sure I agree with this. The jQuery NuGet package hasn't been individually deprecated, and going the route that the code is already using provides more granularity. There's no clear benefit of using this over the jQuery NuGet package other than that a user might prefer it. I find the AspNet.ScriptManager.jQuery package to be fairly unnecessary as it ships with an assembly (AspNet.ScriptManager.jQuery.dll) which essentially performs the same function as the code already in Juice.

Rather than saying Juice should outright use AspNet.ScriptManager.jQuery, I'd say it'd be more prudent to allow for a switch given the broken behavior.

@rustd I'd also recommend looking at the source for Juice to see how the reference is being used. You can override the version set in that reference within your own project to prevent the error you're reporting. Sorry your issue took so long to get a reply. Ping the folks at @appendto (http://appendto.com) who currently maintain it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants