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

Adding bootstrap javascript #93

Closed
l4u opened this issue Apr 9, 2015 · 7 comments
Closed

Adding bootstrap javascript #93

l4u opened this issue Apr 9, 2015 · 7 comments

Comments

@l4u
Copy link

l4u commented Apr 9, 2015

What's the recommended way to load jQuery and Bootstrap JavaScript?

@koistya
Copy link
Member

koistya commented Apr 9, 2015

Bootstrap related files are going to be removed from this starter kit soon. You can use react-bootstrap as a replacement. Though keep in mind, while Bootstrap is an excellent solution for prototyping your initial project, it's generally not a good idea to use it in real web application projects.

@l4u
Copy link
Author

l4u commented Apr 9, 2015

Thanks!

@l4u l4u closed this as completed Apr 9, 2015
@rstormsf
Copy link

@koistya Why bootstrap is not a good idea to use in real web apps?

@korczis
Copy link

korczis commented Jul 9, 2015

Maybe you should think of this @koistya. This is not first request - #138

@9gix
Copy link

9gix commented Jul 24, 2015

After looking at one of the grid css class that seems to have a similar class naming as bootstrap col-sm-4 at here, then I wonder why the index page doesn't show 3 columns instead. And realize that bootstrap wasn't included in the starter kit.

I use bootstrap as prototyping functionality. When it comes to building real application, I prefer not to use bootstrap because it is bloated and customization is not easy in my opinion.

I also think it is good to keep bootstrap as a separate part, because we may have a different css framework choice, such as Zurb Foundation.

@koistya About the col-sm-4 class, because it doesn't affect anything, do we need a grid system in the starter kit to make the column work? or perhaps we can remove the col-sm-4 class? (or it seems not critical perhaps it can be ignored at the moment)

@koistya
Copy link
Member

koistya commented Jul 24, 2015

@9gix good point, need to remove these col-sm-4 etc. classes to avoid confusion. Another idea is to convert static pages from Jade to Markdown.

@nickeblewis
Copy link

That explains that then! I was asking about this on Gitter

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

No branches or pull requests

6 participants