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 3 support #417

Closed
levraipixel opened this issue Feb 16, 2014 · 17 comments
Closed

Bootstrap 3 support #417

levraipixel opened this issue Feb 16, 2014 · 17 comments

Comments

@levraipixel
Copy link

Hi there,

First of all, thanks for this amazing project: it's wonderful.
I was thinking about creating a bootstrap-3-compatible branch: is it something you guys are already working on ? If not, should I fork from the 1.10 branch or from the 2.0 ?
Best

@GBH
Copy link
Member

GBH commented Feb 16, 2014

No, nobody is secretly working on bootstrap 3 compatibility just yet. CMS actually doesn't use bootstrap grid system. However form builder needs updating. See: https://github.com/comfy/formatted_form
FormattedForm needs to be adjusted to use BS3 markup... and its version bumped to 3.0.

If you want to give it a stab that would be awesome.

@levraipixel
Copy link
Author

Still, there are some buttons here an there that need a btn-default class, like in https://github.com/comfy/comfortable-mexican-sofa/blob/master/app/views/admin/cms/pages/index.html.haml for instance.
I'll take a look at the form builder but it seems like a not-so-tiny task to adapt it to BS3.

@GBH
Copy link
Member

GBH commented Feb 16, 2014

Yeah, form builder is the 90% of the work. Should be pretty straightforward though. It's just changing markup in like 2 partial files. Let me know if you have any questions.

@levraipixel
Copy link
Author

like _field_block and _field_inline ?

@GBH
Copy link
Member

GBH commented Feb 16, 2014

Yeap, those are the ones.

@levraipixel
Copy link
Author

Ok I'll give it a try, then.

@hesham
Copy link
Contributor

hesham commented Feb 26, 2014

Is there anyone working on this?

@Emerson
Copy link
Contributor

Emerson commented Feb 26, 2014

Also interested in seeing this happen.

@levraipixel
Copy link
Author

I might work on it, but not before a week or two from now

@gretel
Copy link
Contributor

gretel commented Feb 27, 2014

uh i did think about using bootstrap 3 with cms - so thats not compatible for now?
@GBH, any alternative approaches you can think of? my reason for bootstrap 3 is that designer like it. and as designers generally like CMS.. that seems lika good catch to me.

@Emerson
Copy link
Contributor

Emerson commented Feb 27, 2014

@gretel - right now we're using Bootstrap 2.3, but are looking forward to upgrading to Bootstrap 3. It's a fairly trivial task, we just haven't gotten around to it yet. If there is no movement on this soon I'll look into it myself.

@gretel
Copy link
Contributor

gretel commented Feb 27, 2014

ah ok so i can just go ahead and either wait or commit myself :) thanks

@mrwokkel
Copy link

mrwokkel commented Mar 7, 2014

This is a great opportunity to split the bootstrap css from the comfy specific css.
Maybe even allow for variables (scss)

@Emerson
Copy link
Contributor

Emerson commented Apr 15, 2014

Update:

@GBH has been looking into this a little more and it turns out that it's not as easy as we first believed. Our current form helper might need to be swapped out for something more bootstrap friendly.

@rubylibre - agreed. The more separation the better.

@Emerson
Copy link
Contributor

Emerson commented May 3, 2014

Bootstrap 3 is almost ready for prime time. You can check it out on the 1.12 branch. I've been using it on my project and things mostly seem OK.

You'll likely need to update your routes and controllers to account for the updated namespace that we are now using.

@GBH
Copy link
Member

GBH commented May 4, 2014

Everything is in master branch now. Added dependency on bootstrap-sass and bootstrap_form replaces old formatted_form.

@GBH GBH closed this as completed May 4, 2014
@gretel
Copy link
Contributor

gretel commented May 5, 2014

development looks fine.. now deploying.. thanks!

tom hensel
office +49-4022869965
mobile +49-15735234208

On 04.05.2014, at 19:30, Oleg notifications@github.com wrote:

Everything is in master branch now. Added dependency on bootstrap-sass and bootstrap_form replaces old formatted_form.


Reply to this email directly or view it on GitHub.

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

6 participants