-
Notifications
You must be signed in to change notification settings - Fork 12k
chore(docs): add documentation showing how to include bootstrap #3425
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
Following this updated doc to integrate Bootstrap with angular-cli results into following module conflicts:
Details errors are as below: **ERROR in multi scripts ERROR in multi scripts I'm using below configuration: Any help in this matter would be really appreciated! Thanks, |
@Meet91 that is because this document shows how to install the current bootstrap version, which is Bootstrap 3, while the readme shows how to install Bootstrap 4. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Brocco if you want to show Bootstrap 3 instead of 4, then we should modify the e2e test accordingly. Or even better, add a new one.
Could you make sure the version is mentionned in the doc? |
ea2c866
to
c3e3b70
Compare
Updated to include information for both version 3 and version 4 |
LGTM. Let's see Filipe's review. |
@filipesilva can you re-review this for me? |
@Brocco bootstrap4 also needs a couple of scripts as listed in https://github.com/angular/angular-cli#global-library-installation:
I think bootstrap3 also needs |
To me, there are different levels of how to utilize bootstrap. I was just showing how to reference the CSS and not any of the component JS code. There are different ways to handle that from within angular applications. A lot of people don't want to include jQuery or any other libraries and instead do things the "angular way". Because if we show including those scripts then we should also show how to install both ng bootstrap and ng2 bootstrap which I think would be overkill. |
I agree with @Brocco that this guide should not include any 3rd-party JS libs. With that being said, it might be good to emphasize that this guide just includes the style, and refer to the various Angular Bootstrap implementations. |
For the latest Bootstrap 4 w/ SCSS, I use the following to get things working: In styles.scss (other in a file imported into it):
This way the SASS loader can do its thing properly and I get good IDE support. Not sure if it's the best way, but thought it might be helpful for those using SCSS. Update: Thanks Mike for the new PR with SASS info! |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Applies to #2711