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

bootstrap-css-only dependency #649

Closed
omidahourai opened this issue Apr 21, 2015 · 7 comments
Closed

bootstrap-css-only dependency #649

omidahourai opened this issue Apr 21, 2015 · 7 comments

Comments

@omidahourai
Copy link

This dependency breaks some of the custom bootsrtap varialbes I've set, and some default ones in bootstrap-sass-official (dependency somewhere else). I'm trying to figure out what's the best way to deal with this sort of thing, since it could happen in other places with other sub-dependencies in bower. How do people normally deal with it? If I simply remove the dependency, I have to remember to do it whenever I re-install my bower components.

@SimeonC
Copy link
Collaborator

SimeonC commented Apr 21, 2015

I don't remember that coming up before but there's a request out for removing it so if you want to move all the css code we use into our css file from bootstrap and submit it as a PR removing the dependency that would fix it.

@frankthelen
Copy link

I have exactly the same problem...

The "bootstrap-css-only" dependency from textAngular is overwriting some of the Bootstrap customizations in my project (based on "bootstrap-sass-official"). And I don't have any chance to opt-out the "bootstrap-css-only" dependency coming from textAngular -- Bower is pulling it into my project automatically.

I vote for removing this dependency from textAngular, This lets people chose the Bootstrap package they actually need, e.g., "bootstrap", "bootstrap-sass-official", "bootstrap-css-only", or whatever.

I have seen this in a couple of other projects/packages.

Thanks a lot,
Frank

@pbassut
Copy link
Contributor

pbassut commented Apr 28, 2015

+1

@pbassut
Copy link
Contributor

pbassut commented Apr 28, 2015

@frankthelen Just removing the dependency from bower will only break the plugin considering that it still needs the dependency.

@frankthelen
Copy link

Yes, right, you have to add the Bootstrap package of your choice to your direct project dependencies.

@pbassut
Copy link
Contributor

pbassut commented Apr 28, 2015

I thought the idea was to remove the dependency and create a brand new css style to textAngular. Thus, not conflicting with any other bootstrap or custom style. Adding prefix like '.ta-bootstrap' to css classes would be a good way to go.

Because some people like myself don't want to use bootstrap at all.

Guess I was wrong though.

But still, removing the dependency will make people that just heard of the library to get confused about make it work out of the box.

@frankthelen
Copy link

If you don't want to use Bootstrap at all, adding "bootstrap-css-only" to your direct project dependencies should work.

Making textAngular not depending on Bootstrap at all is a valid approach too. Sure.

But if you use Bootstrap, you want, e.g., that the textAngular Buttons inherit the properties of your customized Bootstrap, for example, the button border radius. Otherwise textAngular will conflict with your Bootstrap theme.

SimeonC pushed a commit that referenced this issue May 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants