Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Split variables from main font awesome less file #860

Closed
wants to merge 2 commits into from

4 participants

@fabsor

When you work with less and use font awesome, the current way of doing things is to download font awesome, change the font path variable in the main file and then include it. This works fine until you want to upgrade to a new release, when you will have to remember to patch the original source file with the fonts folder variable. This can be problematic, especially if you are using a package manager like bower to manage your dependencies automatically.

This pull request moves the font variable to a variables.less file, and adds a new less file, font.less, that can be included in custom projects.

The font-awesome.less file works exactly as it did before.

This is not a problem in the sass version of the project, since sass has the !default feature.

The new workflow when including font awesome in your less project would now be:

@FontAwesomePath:   "path-to-font";
@import "font.less";
@timbrandin

Smooth, solves my problems.

@gustavohenke

That's not needed, LESS will not replace the variables as they reach 'em - variable replacement is one of the last things in the LESS workflow.

So, doing the following would work:

@import "font-awesome.less";
@FontAwesomePath: "path-to-font";
@davegandy
Owner

This should be in 3.1.0-wip branch. And in the future, please submit pull requests to *-wip branches.

@davegandy davegandy closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 17, 2013
  1. @fabsor
  2. @fabsor

    Move stuff around a bit

    fabsor authored
Something went wrong with that request. Please try again.