Skip to content

chore(readme): update Bootstrap CSS inclusion example #2197

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

Closed

Conversation

peterblazejewicz
Copy link
Contributor

This commit changes README example of including Bootstrap dependency
in generated project in that way that Bootstrap CSS dependency is
moved up - to be the first in build bundle.
The reason behind this change is that Bootstrap comes with reset
CSS code and globals that are intended to be global unless reset
and changed by cascading file - here style.css.

Order of CSS styles inclusion:
Before:

- style.css
- bootstrap.css

after:

- bootstrap.css
- style.css

Thanks!

@filipesilva
Copy link
Contributor

Fair point. Marking to be merged.

This commit fixes README example of including Bootstrap dependency
in generated project in that way that Bootstrap CSS dependency is
moved up - to be the first in build bundle.
The reason behind this change is that Bootstrap comes with reset
CSS code and globals that are intended to be global unless reset
and changed by cascading file - here style.css.
Thanks!
JJayet pushed a commit to JJayet/angular-cli that referenced this pull request Oct 2, 2016
This commit fixes README example of including Bootstrap dependency
in generated project in that way that Bootstrap CSS dependency is
moved up - to be the first in build bundle.
The reason behind this change is that Bootstrap comes with reset
CSS code and globals that are intended to be global unless reset
and changed by cascading file - here style.css.
Thanks!

Close angular#2197
deebloo pushed a commit to deebloo/angular-cli that referenced this pull request Oct 3, 2016
This commit fixes README example of including Bootstrap dependency
in generated project in that way that Bootstrap CSS dependency is
moved up - to be the first in build bundle.
The reason behind this change is that Bootstrap comes with reset
CSS code and globals that are intended to be global unless reset
and changed by cascading file - here style.css.
Thanks!

Close angular#2197
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants