-
Notifications
You must be signed in to change notification settings - Fork 5
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
replace easily-breakable fontawesome icons with inline svg instead. #24
Comments
The icon fonts are breaking. |
See
for a basic example of how it's done in bluegenes. |
Hi, I am an Outreachy applicant and would like to contribute. |
Glad to hear! I have assigned you the issue. |
Thanks so much. Request you to help me set up the project in my local machine. Is there any documentation available? Also, please could you let me know how to replicate the issue |
I have set up my windows machine. I need some clarifications about the issue.
|
Sorry for the delayed response; we have been swamped by communications from Outreachy applicants. Follow the README to run this project. You will need node, npm, leiningen and Java (eg. openjdk).
At the top of https://github.com/intermine/bluegenes/blob/dev/src/cljs/bluegenes/components/icons.cljs you will find instructions on how to attain similar icons from Icomoon to replace the ones imported with fontawesome. Then create a similar icons.cljs file in this repo, with the svg converted to hiccup, and reference it at the top level like done in https://github.com/intermine/bluegenes/blob/dev/src/cljs/bluegenes/views.cljs#L70. Lastly, replace all uses of fontawesome in im-tables-3 to use the svg instead, like so It is a bit of a difficult and involved task, but you are welcome to tackle it if you wish! |
Thanks for the response. I can now understand the changes needed. I would like to know if I need to get the equivalent of all the icons present in font-awesome.css file? And when you say this project is used by Bluegenes, does that mean these icons are used in Bluegenes project and not in im-tables? Kindly confirm |
You only need to add Icomoon equivalents of the icons that are currently used. font-awesome icons are used like
These icons are used in im-tables-3 (this project), which is used by Bluegenes. |
icons should be svg. svg is trusty. icon fonts invariably break at some point.
Example:
by contrast, inline svg is embedded in the page and can't suffer from CORS
The text was updated successfully, but these errors were encountered: