-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Bootstrap v3 is not compatible with jQuery 3.0 #16834
Comments
We should update our docs and package metadata to reflect this incompatibility. |
I think things like this are a good reason to look into issue #15068 again. |
We still have absolutely zero plans on that front. 😄 |
In my opinion, it would make more sense to attempt for compatibility with jQuery 3.0 after it is out of the alpha stage and it can confidently be developed against in regards to general API stability. I assume the big pieces are done now (based on what their announcement said) but it sounded to me as though more changes will probably occur prior to beta / GA. If someone know's it's stable, it probably shouldn't be anything crazy to add compatability. My only initial concern (without looking at what is necessary) that it may require some duplication in areas if there isn't a possible cross version solution and |
@kkirsche jQuery 3 is more strict than jQuery 2. It's like XHTML and HTML. Backward compatibility is preserved and code doesn't look like a mess. |
As jQuery 3.0 is a replacement for both jQuery 1.x and 2.x, I would say that this has to get fixed in Bootstrap 3 and not wait until Bootstrap 4 (Because how long is that going to take? In October 2014, it was said that the first pre-release would land in a couple of weeks, we're now 9 months further). |
@Studio384 jQuery 3.0 is only alpha still and is not a stable release. |
…closes #16834 [skip sauce] [skip validator]
Can someone summarize the problems you've encountered with jQuery 3.0 alpha and Bootstrap 3, in the form of a few simple jsbin examples? I'd like to see what it takes to fix this. |
@dmethvin Modals seem to break due to the |
jQuery dropped the new .show() and .hide() methods in the latest beta (3.0.0-beta1). Modals and other components seem to work OK with it. |
The 3.0.0-beta1 still does change the way show/hide work but it tries to force fewer layouts in the process. |
@mdo @cvrebert jQuery 3 RC1 is here.
JUnit bootstrap tests with included jQuery 3 RC1 + jQuery Migrate plugin: I think it's possible to fix the listed issues And show() and hide() methods are not an issue anymore
|
The scrollspy failure should be fixed by #19906. That was the only failure I saw in the v4-dev branch. @Maxkorz what branch was used for those tests? A selector of The one with SVG positioning may be related to jquery/jquery#2895? What browser was used for those tests? |
@dmethvin I used the master branch.
This issue is about Bootstrap v3.
Selector
selector = $this.attr('href') so simple HTML
I used Chrome 51 to run the tests. |
I would recommend a new issue instead of reviving this with a clear note whether this is regarding V3, V4 or both. |
Now that jQuery 3.0 is officially released I think this needs to be revisited. Is Bootstrap 3.x going to updated to work with it or not? Given that there appears to be relatively few issues it would be a shame for Bootstrap 3.x users to be stuck with an old jQuery dependency (especially since Bootstrap 4.x is not even past the alpha stage--so who knows when it will actually be released). |
… preserve behavior under jQuery 3 This code ought to have used .prop instead of .attr in the first place, but we can't get rid of the attr manipulation now due to backward compatibility constraints. Refs https://github.com/jquery/jquery-migrate/blob/3.0.0/warnings.md#jqmigrate-jqueryfnremoveattr-no-longer-sets-boolean-properties Refs #16834 [skip validator]
… preserve behavior under jQuery 3 (#20278) This code ought to have used .prop instead of .attr in the first place, but we can't get rid of the attr manipulation now due to backward compatibility constraints. Refs https://github.com/jquery/jquery-migrate/blob/3.0.0/warnings.md#jqmigrate-jqueryfnremoveattr-no-longer-sets-boolean-properties Refs #16834 [skip validator]
@cvrebert IIRC they pulled the formerly disruptive change in behavior of |
@fuhrysteve There are still some breaking changes in the final release, though they're less drastic: http://jquery.com/upgrade-guide/3.0/#breaking-change-show-hide-and-toggle-methods-now-respect-more-stylesheet-changes |
Both blockers have been fixed. Now we just need to update the package metadata and the version check code. |
Refs #16834 [skip sauce] [skip validator]
What's the status of this issue? |
@niemyjski, it is done. Bootstrap 3.3.7 now supports jQuery 3+. |
…#20338) Closes twbs#16834 [skip validator]
jQuery 3.0 and jQuery Compat 3.0 Alpha Versions Released
I have not tested everything but Modals are broken because of this major change:
JSBin jQuery v1.11.1 http://jsbin.com/sokiyuzepi/1/edit?html,output
JSBIN jQuery v3.0.0 alpha http://jsbin.com/xuteferejo/1/edit?html,output
The text was updated successfully, but these errors were encountered: