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

Add "dist/css/bootstrap.css", to bower.json > main #16659

Closed
wants to merge 1 commit into from

Conversation

alber70g
Copy link

It's useful when you use gulp wiredep to have css (when needed) being injected

It's useful when you use gulp wiredep to have css (when needed) being injected
@kkirsche
Copy link
Contributor

@alber70g this change was due to bower changing it's spec for main and is not a bug with bootstrap. See: #16359

@alber70g
Copy link
Author

This means I have to compile a third party library, instead of use it? It's an entrypoint for when you want to use the library, and not the source. It's not to be instead of it, but aside of the existing .less file.

@mdo
Copy link
Member

mdo commented Jun 17, 2015

Let's discuss a bit more in #16663 to see if we want to say screw the spec here.

@redysanin
Copy link

Even we also face same issue in bootstrap 3.3.5 . at grunt serve it remove

"bower_components/bootstrap/dist/css/bootstrap.css"
It look like they remove "dist/css/bootstrap.css" from bower_components\bootstrap\bower.json

We fix this issue temporary by overrides our project bower.json. for us it work fine. but waiting for some solution from bootstrap.

"overrides":{
"bootstrap" : {
"main": [
"less/bootstrap.less",
"dist/css/bootstrap.css",
"dist/js/bootstrap.js"
]
}
}

@cvrebert cvrebert added the meta label Jun 17, 2015
@cvrebert
Copy link
Collaborator

Closing this in favor of more discussion in #16663.

@cvrebert cvrebert closed this Jun 18, 2015
@twbs twbs locked and limited conversation to collaborators Jun 18, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants