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

Broken against jQuery 3 #1948

Closed
pugnascotia opened this issue Jul 17, 2016 · 22 comments
Closed

Broken against jQuery 3 #1948

pugnascotia opened this issue Jul 17, 2016 · 22 comments

Comments

@pugnascotia
Copy link

pugnascotia commented Jul 17, 2016

Expected behaviour

Contents of picker widget e.g. days of the month should be displayed.

Actual behaviour

Picker opens but contents are not displayed

Datepicker version used

1.6.1
Broken against jQuery 3.0.0 and 3.1.0

Example code

http://jsfiddle.net/4x9wfayw/

@bcrigler
Copy link

ditto

@glen-84
Copy link

glen-84 commented Jul 22, 2016

Error: Bootstrap's JavaScript requires jQuery version 1.9.1 or higher, but lower than version 3

@pugnascotia
Copy link
Author

If I'm reading this thread correctly, Bootstrap have addressed this:

twbs/bootstrap#16834

However it's targeted at v3.3.7, which isn't available yet.

@hakuno
Copy link

hakuno commented Jul 26, 2016

Okay, Bootstrap 3.3.7 is released. And Datepicker 1.7.0 isn't working. Any help? I can't get it working with jQuery 3.1.0 right now.

@joantorres
Copy link

Same problem here. Any workaround other than going back to jQuery 2?

@lucasmm
Copy link

lucasmm commented Jul 27, 2016

for now i'm using this in bower.json

"bootstrap-datepicker": "https://github.com/eternicode/bootstrap-datepicker.git#e48ea20"

e48ea20 is commit where correction has been made.

@glen-84
Copy link

glen-84 commented Jul 27, 2016

jQuery migrate shows the following:

JQMIGRATE: jQuery.unique is deprecated, use jQuery.uniqueSort

@acrobat
Copy link
Member

acrobat commented Jul 27, 2016

I should be able to make some time next week to look into this issue

@acrobat acrobat added the is-bug label Jul 27, 2016
@rhpot1991
Copy link

Bit by this when upgrading NuGet packages. Previously Bootstrap didn't support jQuery 3, but now that they do I am free to upgrade. I get an empty date picker when jQuery 3 is in place.

@evanleck
Copy link

Bootstrap v4 just added support for jQuery 3 with the alpha 3 milestone.

Running against that, I see an issue with display: none on .datepicker-months, datepicker-years, datepicker-decades, and datepicker-centuries. They all remain hidden even once div.datepicker is shown.

terop added a commit to terop/ktra-indexer that referenced this issue Jul 31, 2016
The component does not work with jQuery >= 3 yet. Need to check for a
new version once in a while. See
uxsolutions/bootstrap-datepicker#1948.
@acrobat
Copy link
Member

acrobat commented Aug 3, 2016

It seems that #1964 fixes the jquery3 issue. Can you guys test and confirm the patch?

@acrobat acrobat added the has-pr label Aug 3, 2016
@glen-84
Copy link

glen-84 commented Aug 3, 2016

@acrobat #1958 looked like a better (and cleaner) fix to me, but I don't have time to test it right now.

@evanleck
Copy link

evanleck commented Aug 3, 2016

@glen-84 @acrobat I've updated my code with the fix from #1958 and it works fine for me.

@acrobat
Copy link
Member

acrobat commented Aug 3, 2016

PR #1958 was merged in master and the fix is backported to 1.6 in a923be2. A new version (1.6.2) was pushed and published

@priley86
Copy link

priley86 commented Aug 4, 2016

@acrobat hi, can you explain why jQuery.unique was left in, and we have not merged jQuery.uniqueSort? uniqueSort is supported by jQuery 1.1.2+ . This appears to break the treeview for jquery-3.0.

@acrobat
Copy link
Member

acrobat commented Aug 4, 2016

I overlooked that extra fix, will make some time and change the method and push a new release!

@acrobat
Copy link
Member

acrobat commented Aug 4, 2016

@priley86 Can you test commit df4542c if this is enough for your issue?

@priley86
Copy link

priley86 commented Aug 5, 2016

@acrobat +1 this will resolve this issue

@acrobat
Copy link
Member

acrobat commented Aug 5, 2016

I've pushed a new release with the uniqueSort fix! https://github.com/eternicode/bootstrap-datepicker/releases/tag/v1.6.3

@rhpot1991
Copy link

Anyone know how this gets pushed to NuGet?

@acrobat
Copy link
Member

acrobat commented Aug 11, 2016

@scott-xu maintains the nuget package! Ping @scott-xu

@scott-xu
Copy link

pushed

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

No branches or pull requests