Skip to content
This repository has been archived by the owner on Dec 23, 2021. It is now read-only.

Updated bootstrap to 2.0.3 #1

Closed
wants to merge 2 commits into from
Closed

Updated bootstrap to 2.0.3 #1

wants to merge 2 commits into from

Conversation

buzzedword
Copy link

Added LESS library for complete reference. Gradual conversion of style.css to codeigniter-style.less, using mixins for ease of use.

Updated plugins to use 2.0.3 versions, included all for reference.

Updated header.php to point to LESS compiled CSS, as well as concatenated JS.

Daniel Garcia added 2 commits May 9, 2012 15:29
Signed-off-by: Daniel Garcia <danny@noisenewyork.com>
Signed-off-by: Daniel Garcia <danny@noisenewyork.com>
@sjlu
Copy link
Owner

sjlu commented May 10, 2012

I don't think less source files should be added here, though the update to 2.0.3 will be necessary.

@buzzedword
Copy link
Author

I'd disagree with that, only because an increasing number of bootstrap
resources are .less dropin files-- considering bootstrap was built
less first. For example, themes on WrapBootstrap override the
variables.less file, and add a bootswatch.less file, with no CSS
options. While its up to the dev to decide to use less or the provided
CSS source, the option should be there.

On May 9, 2012, at 11:22 PM, Steven Lu
reply@reply.github.com
wrote:

I don't think less source files should be added here, though the update to 2.0.3 will be necessary.


Reply to this email directly or view it on GitHub:
#1 (comment)

@sjlu
Copy link
Owner

sjlu commented May 10, 2012

I think this is a good extra feature to have, but a proper LESS compiler or at least bash script which compiles LESS should be created before a pull should be done.

@bond-
Copy link

bond- commented Oct 25, 2012

+1 for the idea of a less compiler. I'm not much of a php programmer but yii has it. Can it be ported to CI??

@sjlu
Copy link
Owner

sjlu commented Oct 25, 2012

No worries, I'll be handling this in the next version, which will be able to compile from a series of repositories. More info later!

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