Use bootstrap styling by default about angular-form-for HOT 6 CLOSED

Hey @epicbagel. Thanks for sharing your thoughts.

I shied away from using Bootstrap it's pretty opinionated- which can be great but the past couple of places I've worked haven't been a fan of its default styling and so we've steered clear. (It can be a hassle to reset parts of Bootstrap's styling.) I didn't want to force people to rely on Bootstrap unnecessarily.

I did avoid any kind of nasty inline styling so that folks could choose to import my stylesheets or write their own. (I mentioned this briefly in the template overrides section). I also include pre-compiled styles in the dist folder of course, so you don't have to fit Stylus (or Sass) into your pipeline just to compile formFor's styles.

That being said, I'd be open to providing a secondary set of stylesheets that build on top of plain, vanilla Bootstrap styles- so that Bootstrap users like yourself could just include them instead.

from angular-form-for.

One suggestion would be to use Bootstrap's markup by default, so people hook into that out the box. The styl files you've included could then just be migrated to hook into the bootstrap css names and be included when people don't have or want bootstrap?

from angular-form-for.

That's a good suggestion. Let me address that in a point release. :)
Thanks!

from angular-form-for.

Hi again @epicbagel! Just writing to let you know that your suggestion has been implemented as part of Release 1.1.0.

Here's a Plnkr to let you compare Bootstrap-only styles vs formFor custom styles:
http://plnkr.co/edit/jY8TC8xbMdK5dBJMResL?p=preview

Thanks again for the suggestion!

from angular-form-for.

@bvaughn amazing, I think it will certainly help add traction to the project if people can drop it in without hooking in manual styles. Will be using this today on a prototype so will feedback with suggestions.

It may be worth updating the docs to list bootstrap is supported out the box or you can use custom styling.

from angular-form-for.

Good suggestion. I've updated the wiki this morning.

I'll check into your other issue sometime this afternoon. :)

from angular-form-for.