-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Fix peerDependency build issue & change size() to length #1726
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
Conversation
…lock to widget. original credit goes to @freemer
looks good to me, hopefully it can be merged and a new build done |
I just commented on the original ticket. This doesn't actually work as intended, the date time picker pop up is blank. Will cancel and investigate further. |
ah okay :( |
Ugh, it was my own environmental issues. Going through an upgrade and had some old CSS leaking through that prevented the calendar picker from displaying. Re-opening, sorry for that. |
least it works thats all that matters, good work, not sure how you get it signed off to be deployed |
@Eonasdan can this be scheduled for a merge |
Hi @Eonasdan, this fixes an issue where jQuery 3.0 was not supported. I just read your contributing guide, please let me know if you wish me to switch this PR to the development branch. As an aside, what is the release schedule? We depend on this project via NPM and it looks like a release hasn't occurred since 2015. Regards, |
@tsneed290 i think the pull request needs to point to development not master
|
Thanks @steveyj79, will move it. |
@steveyj79 it looks like the development branch has the fixes needed to support jQuery 3. While not desired, I am able to depend on the project in my package.json by referencing the git commit hash:
@Eonasdan When do you think you'll be releasing 4.17.42 to NPM's registry? |
dev is now master and updated in npm |
Cheers @Eonasdan |
Thanks @Eonasdan!! |
@Eonasdan, since this is a hotfix, could you please replace the buggy one currently at NuGet? |
Can this be updated in bower please? bower still shows and pushes version 4.17.37 as the latest which uses input.size. |
Wouldn't creating a |
@Eonasdan or @nikoskalogridis can one of you guys please create a new tag so we can get the changes via bower? |
This actually appears to be a real issue in jQuery 3.1 that has been reported here #1765. |
For bower users who are not aware, there is version 4.17.45 which works. |
This fixes the build issue as part of #1664.