It would be really nice if this theme is integrated with color module where you can change the color of header strip and other elements.

Comments

frankbaele’s picture

I personally think this is not a good solution because the color module is not a good solution

frankbaele’s picture

what works well is the sweaver module, what i prefer far more then any other client color picking solution

http://drupal.org/project/sweaver/

valor’s picture

Yes I agree with you.

frankbaele’s picture

Status: Active » Closed (won't fix)
andregriffin’s picture

Project: Twitter's Bootstrap » Bootstrap
vlooivlerke’s picture

Issue summary: View changes
Status: Closed (won't fix) » Active

Hi, I am opening this issue again. When you state "colour module is not a good solution" what do you mean?

In my use case I use domain access with domain theme module. If you have a theme with colour module enabled you can give each domain its own theme colour as the variables is stored per domain. This is an easy way to give each domain its own colour pallet.

Or will the colour module affect the performance of bootstrap. Then it is not a good solution.

How about using these sites for example, a site like http://www.lavishbootstrap.com will let you create a custom css from an image (color scheme)

or paintstrap is even better, as it uses kuler: http://paintstrap.com

Or would the only solution be to have 100's of subthemes to choose from in a domain access setup?

andregriffin’s picture

It would likely be best to use a subtheme with your color variation for each domain. Either using a the third-party bootstrap building site, or just changing the LESS variables and recompiling the stylesheets.

vlooivlerke’s picture

Ok thanks. I noticed that the colour module will not be able to change the LESS values. So I think we can clearly state that the colour module is no solution for bootstrap.

iaminawe’s picture

This video shows how to use the less module as a bridge to allow this if its what you need to do
https://www.youtube.com/watch?v=r8vVfmQLmAA

I found this theme has a simple implementation with the color module not using less
https://drupal.org/project/bootstrap-business

markhalliwell’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev

Yes, I personally know Kyle and saw his presentation on this very topic at our last http://www.meetup.com/North-Texas-Drupal/. It should be interesting to see how we can actually integrate this directly into the base-theme. I'll ping him this thread so he can get involved with this discussion.

kyletaylored’s picture

Totally, I'd be happy to help and see if we can. It might be more suitable for Bootstrap UX, but either way I think we can get something started.

iaminawe’s picture

Also cross linking with this relevant feature request in bootstrap_ux - https://www.drupal.org/node/2129353

I have been attempting to do this in a sub theme and have made some progress following along with the code in Kyles video but have found a few errors I am still trying to work through

Notice: Undefined index: palette in color_scheme_form_validate() (line 351 of ***/modules/color/color.module).
Warning: Invalid argument supplied for foreach() in color_scheme_form_validate() (line 351 of ***/modules/color/color.module).
Notice: Undefined index: palette in color_scheme_form_submit() (line 372 of ***/modules/color/color.module).
Warning: implode(): Invalid arguments passed in color_scheme_form_submit() (line 413 of ***/modules/color/color.module).
Warning: Invalid argument supplied for foreach() in _color_rewrite_stylesheet() (line 500 of ***/modules/color/color.module).
Notice: Undefined index: text in _color_rewrite_stylesheet() (line 527 of ***/modules/color/color.module).
Notice: Undefined index: link in _color_rewrite_stylesheet() (line 527 of ***/modules/color/color.module).
Notice: Undefined index: link in _color_rewrite_stylesheet() (line 527 of ***/modules/color/color.module).
Notice: Undefined index: link in _color_rewrite_stylesheet() (line 527 of ***/modules/color/color.module).
Notice: Undefined index: base in _color_rewrite_stylesheet() (line 527 of ***/modules/color/color.module).
Notice: Undefined index: link in _color_rewrite_stylesheet() (line 527 of ***/modules/color/color.module).

iaminawe’s picture

I was able to resolve those issues by making sure I had a a base, text and link variable set for each color palette - all works nicely now.

@kyletaylored - how were you able to view the changed stylesheet without emptying the less cache? When I make the change in settings, there is no current visible change until after I empty the less cache and only then do I see that my updates have worked. I have enabled developer mode in the less module settings which I thought was supposed to rebuild from the less file on each page load.

Thanks for the awesome video series by the way - very helpful stuff.

kyletaylored’s picture

Yep, you definitely want to make sure you at least have the variables defined or it'll throw those errors, which is why I stick with the brand_primary, brand_secondary, etc and then use those to color links, body text, backgrounds, etc.

In the color.inc, I just call less_flush_caches() to make sure the cache gets rebuilt upon saving.

// Flush LESS cache
if (module_exists('less')) {
  less_flush_caches();
}
iaminawe’s picture

Thank you Kyle - that totally does the trick. Much appreciated - its working perfectly.

markhalliwell’s picture

Version: 7.x-3.x-dev » 7.x-4.x-dev
Status: Active » Postponed
kyletaylored’s picture

This was a video I recently made highlighting some integrations I did using LESS and a couple other modules.

Automated Themes using Color Palettes in Drupal

FWIW, i don't think this is something that should be built in, but more or less documented on how to do in a subtheme. With LESS 7.x-4.x, it's much easier to do by dropping variables into an .info file.

markhalliwell’s picture

Heh, thanks for the shout out.

I'm still kind of optimistic that we can, one day, support this out of the box. Color/LESS can be quite confusing for most and anything we can do to "officially" support it would nice. That's why I'm postponing this for now. Definitely not an active "feature".

Kristina Katalinic’s picture

@kyletaylored that is some really awesome work there. Can you point to the code repository for enterprise bootstrap theme from your video in #17? Thanks

markhalliwell’s picture

Version: 7.x-4.x-dev » 8.x-4.x-dev

Changing to proper branch version for #2554199: Bootstrap 4.

markhalliwell’s picture

Status: Postponed » Closed (won't fix)

I don't think this is going to happen. Re-open if, and only if, there's real progress to be made (aka patch).