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

feature: copy as "SVG Markup" syntax (embeddable in HTML) #63

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

grimen
Copy link
Contributor

@grimen grimen commented Jan 11, 2016

Quickly queried and transpiled from embedded *.svg fonts. A generic
approach that works with any icon fonts without extra work. Additional
requirement: Adds library to each icon instance, makes sense for
other reasons too.

Quickly queried and transpiled from embedded *.svg fonts. A generic
approach that works with any icon fonts without extra work. Additional
requirement: Adds `library` to each icon instance, makes sense for
other reasons too.
@grimen
Copy link
Contributor Author

grimen commented Jan 11, 2016

fixes #49

@grimen
Copy link
Contributor Author

grimen commented Jun 22, 2016

Status on this?

@thomaspark
Copy link
Owner

thomaspark commented Jul 1, 2016

Tested it out and it's working good overall. With the conflicts cleaned up: https://github.com/thomaspark/glyphsearch/tree/grimen-feature-copy-svg-markup

However, there is an issue with Material Design Icons. In the SVG file MaterialIcons-Regular.svg, each glyph actually has its unicode attribute filled with the glyph-name. Once this is corrected in their library, this should be good to go.

screen shot 2016-07-01 at 12 17 54 pm

@grimen
Copy link
Contributor Author

grimen commented Jul 11, 2016

OK, should be a minor fix but will see when I got time too look at it because deadlines and summertimes. :)

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

2 participants