Converts icon variables to use a map and adds a new function and mixin to make usage easier.
This could use some discussion, but: this PR moves the icon variable declarations to /variables/icons, since they don't really make sense in defaults anymore (they're essentially core pieces). Likewise, the two specific icon declarations have been moved into a new module/icons.
I did leave the actual defaults for icons in base/defaults, though I renamed the existing icon() mixin to icon-config(), since that's all it is and it can get changed to a placeholder once we drop IE8 support.
The icon function and mixin both are largely the examples referenced in #118 from Eric Suzanne, though I've added some optional variables to the mixin both for which pseudo element and which icon map to use. I also added a warning message to the function similar to what we have in the color() function.
I've also updated the few instance of existing icon font variable usage to use the new mixin.
elseloop
commented
9 years ago 
Converts icon variables to use a map and adds a new function and mixin to make usage easier.
This could use some discussion, but: this PR moves the icon variable declarations to
/variables/icons, since they don't really make sense in defaults anymore (they're essentially core pieces). Likewise, the two specific icon declarations have been moved into a newmodule/icons.I did leave the actual defaults for icons in
base/defaults, though I renamed the existingicon()mixin toicon-config(), since that's all it is and it can get changed to a placeholder once we drop IE8 support.The icon function and mixin both are largely the examples referenced in #118 from Eric Suzanne, though I've added some optional variables to the mixin both for which pseudo element and which icon map to use. I also added a warning message to the function similar to what we have in the color() function.
I've also updated the few instance of existing icon font variable usage to use the new mixin.