-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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
New arXiv favicon #1704
Comments
If the icon is already showing somewhere, but it is just not the updated one, all you need to do is update the version of academicon used in _config.yml |
Thank you! Is this because I also need to change the integrity below version? Thank you! |
Yes, you also have to update the integrity. The integrity is the hash of the file. With the wrong integrity the file will not load at all. |
As you can see here, the latest academicons npm package is For But, the new arXiv icon isn't there as well. I have tested the 1.9.4 locally by manually installing.
I don't know how much this will hamper the site speed. @george-gca Is it good to install the fonts locally or call from the cdn? |
Apparently it's been a while since the npm package was updated. Also there are still missing icons from academicons that could be added (I created a feature request for one a while ago). One day I will create a PR for this. According to this site, it would be better to load things from CDN since it can load from sources closer to the user. At the same time, loading locally can offer other benefits when used with some other solutions like this one here. I will try to do some more thorough tests with the |
EDIT: I just realized that I did include I would for now recommend locally loading the academicons package if you want the latest version. If someone could send a PR for this would be great. |
I believe this was fixed in #1562 |
Hi there!
This is not really an issue.
I want my site to display the new arXiv favicon [https://jpswalsh.github.io/academicons/].
But I don't know how to modify the settings to make this possible.
Thanks a lot for your help!
The text was updated successfully, but these errors were encountered: