Removed unnecessary usage of the single fa class

[elboletaire]

As all icon classes begin with fa- we can remove the need to use the single fa class.

Before this change you need to define your icons like this:

<span class="fa fa-glass"></span>

With this change, you'll only need to do:

<span class="fa-glass"></span>

[tagliala]

Hi,

this was intentionally done in 4.0.0 in order to avoid universal selectors like

[class^="fa-"],
[class*=" fa-"] { ... }

because of performance.

More info:
http://stackoverflow.com/questions/19407451/how-to-profile-css-selectors (unanswered)
https://gist.github.com/tagliala/7174356

[elboletaire]

From the first link...

Given the speed at which selectors are parsed, you'd be better off focusing on other aspects of the website which could use a performance gain.

As developers we loose more than 40ms each time we need to add the single fa class.

But it's ok, it's your project and your decission. I'll work my fork then... 😄

[aleixfabra]

👍 @elboletaire

[tagliala]

refers to #2454 (and to less/less.js/issues/1485)

with a proper mixin, a fork should not be needed for this purpose

[elboletaire]

A proper mixin work only if I do everything with less.

.my-post {
    .icon(glass);
}

But I apply many icons using only html, that's why I need a fork.

<a class="my-post"><span class="fa-glass"></span> My link</a>

Or are you proposing me anything different? @tagliala