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

Add SVG icons as an option #168

Open
yochannah opened this issue Jan 3, 2019 · 10 comments
Open

Add SVG icons as an option #168

yochannah opened this issue Jan 3, 2019 · 10 comments
Assignees

Comments

@yochannah
Copy link
Member

the current icon solution uses icon fonts, which repeatedly get caught up by CORS issues and result in nonsense icons instead of page navigation controls.

I suggest we upgrade to allow users to choose embedded SVG if they wish, and use the free icons from icomoon: https://icomoon.io/app/#/select

@CodHeK
Copy link
Contributor

CodHeK commented Mar 7, 2019

@yochannah I'd like to work on this project and found this in the list of GSoC'19 projects under InterMine ... Can I take up this issue for a start ...?

@CodHeK
Copy link
Contributor

CodHeK commented Mar 7, 2019

@yochannah I'm having problems setting up the project, the commands I ran successfully :

  1. git clone
  2. npm install
  3. bower install

I am not able to run the make js command as mentioned in the contributing docs...can you please help with setting up a demo project ... Thankyou!

@yochannah
Copy link
Member Author

This was part-fixed in #172 where we realised that the contributing guide was out of date. Thanks for this fix!

@CodHeK As you may have noticed, building this project is a bit tricky! We're hoping a GSoC student will be able to help us upgrade this library to a recent version of node, but right now I have the suspicion it needs to be built under node 0.10 or lower. Some of the notes in the project spec here might help: http://intermine.org/gsoc/project-ideas/2019/detailed-specs/upgrade-imjs-and-imtables

@Manasa2850
Copy link
Contributor

@yochannah I would like to work on this. Could you please guide me on how to start?
Thanks!

@yochannah
Copy link
Member Author

@Manasa2850 Amazing, thank you 🙌

Start by setting up the repo using the contributing guide: https://github.com/intermine/im-tables/blob/dev/CONTRIBUTING.md

Once you've gotten that far, have a hunt about for the icon code and let me know how far you get - if you get stuck we can work things out together! ✨

@km-js
Copy link

km-js commented Mar 6, 2020

Being an outreachy aspirant, I would like to work on this.

@Manasa2850
Copy link
Contributor

@km-js sorry I've been assigned this and I'm working on this :)

@km-js
Copy link

km-js commented Mar 7, 2020

@Manasa2850 Cool. If you need any help let me know 👍

@Manasa2850
Copy link
Contributor

@yochannah I've set up the repo. As I was looking through the codebase I found most of the code related to icons in imtables.js and main.css
I thought of creating something like a button in imtables.js which would enable users to choose between glyph icons or SVG. Then we can replicate whatever we have for glyph icons for SVG icons as well.
Please do let me know if this is the right way to proceed. Thanks!

@yochannah
Copy link
Member Author

so - the intent is not for users to change the icons, but for intermine admins to be able to do so
I can think of two ways. 1 would be to upgrade to modern fontawesome, which I think supports svg now
and the other would be to manually add all the icons needed using something like icomoon.app
if you look here at the options table there is a way to configure icons already, so if you went for the second option, you might add "svg" to the list of icon choices available - see

https://github.com/intermine/im-tables/blob/dev/CONFIGURING.md

and

https://github.com/intermine/im-tables/blob/dev/src/options.coffee#L79

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

No branches or pull requests

4 participants