Skip to content

Distinct Bootstrap jQuery requirement #15073

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

Merged
merged 1 commit into from
Apr 16, 2021
Merged

Distinct Bootstrap jQuery requirement #15073

merged 1 commit into from
Apr 16, 2021

Conversation

ChristianOellers
Copy link
Contributor

Add distinction to jQuery requirement for Bootstrap version 5, which does not require it to be used:
https://getbootstrap.com/docs/5.0/getting-started/javascript/#still-want-to-use-jquery-its-possible

Text is a suggestion, I feel it could be more elegantly written?

@Nyholm
Copy link
Member

Nyholm commented Mar 9, 2021

I agree with this. But Symfony does not support Bootstrap 5 yet.

Add distinction to jQuery requirement for Bootstrap version 5, which does not require it to be used:
https://getbootstrap.com/docs/5.0/getting-started/javascript/#still-want-to-use-jquery-its-possible
@javiereguiluz javiereguiluz changed the base branch from 5.x to 4.4 April 16, 2021 11:12
@javiereguiluz javiereguiluz requested a review from xabbuh as a code owner April 16, 2021 11:12
@javiereguiluz javiereguiluz merged commit 99a5748 into symfony:4.4 Apr 16, 2021
@javiereguiluz
Copy link
Member

Thanks Christian, we've merged this with some minor tweaks.

Tobias, you are right, but that should only affect you when trying to use a Form theme for Bootstrap 5 (there's none yet). But for the normal usage of Bootstrap 5 in Webpack Encore, it's OK to remind them to not use jQuery anymore.

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

Successfully merging this pull request may close these issues.

4 participants