Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Selector optimization #16

Closed
wants to merge 7 commits into from

Conversation

wesleycho
Copy link

This is meant as a fix for #7 - this requires the element to also contain the class ion in addition to the icons. It also has the fix implemented for #11 as well. This is the fix that Font Awesome used.

If support for LESS and/or SASS are not desired, you can easily cherrypick the commit of interest.

@lipis
Copy link

lipis commented Nov 5, 2013

Yes please!

@mlynch
Copy link
Contributor

mlynch commented Nov 5, 2013

We are working on a totally new build that isn't using Icomoon, and with that we will have faster selectors, and also SASS files get generated with the build, so since it's automatic we will probably include them. I'll see about getting LESS in there as well.

I am going to close this mainly because we won't just be merging it in, but all of this will be in the next release of Ionicons coming soon. Thanks guys!

@mlynch mlynch closed this Nov 5, 2013
@lipis
Copy link

lipis commented Nov 5, 2013

Awesome.. and yes.. show some love to LESS as well.. :)

@vjpr
Copy link

vjpr commented Jan 24, 2014

+1 for LESS!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants