Use Fork Awesome as the default icon font, and rework settings
In16e3058, a FONT_AWESOME setting was introduced and used as such: "https://kit.fontawesome.com/{{ FONT_AWESOME }}.js" Also in16e3058, the default way to fetch Font Awesome was changed to Javascript (before we used to load is as CSS). The present commit does several changes: - Revert the default back to CSS (instead of JS), as I think that for a font, there's no need for Javascript. A font is a static asset, so it's enough to load it as a static CSS file, both for privacy and security. Also, think about those who disable JS in their browser. - Remove the `FONT_AWESOME` setting, as it's a bit too limiting to force users into `kit.fontawesome.com`, it doesn't leave much space for configuration. - Instead, introduce two variables `FONT_AWESOME_CSS` and `FONT_AWESOME_JS`, so that users have more freedom to define how they want to load the Font Awesome. - Use Fork Awesome as the default, if no setting is given by user. Meaning that we get back to the `fa` css classes (instead of `fas` or `fab`)
This commit is contained in:
@@ -16,7 +16,7 @@
|
||||
{% for name, link in SOCIAL %}
|
||||
{% if name == 'email' %}
|
||||
<a class="sidebar-nav-item" href="mailto:{{ link }}">
|
||||
<i class="fas fa-envelope"></i>
|
||||
<i class="fa fa-envelope"></i>
|
||||
</a>
|
||||
{% elif name in ['academia', 'acclaim', 'acm', 'acmdl', 'ads', 'arxiv',
|
||||
'biorxiv', 'ceur', 'coursera', 'cv', 'dataverse',
|
||||
@@ -31,12 +31,12 @@
|
||||
</a>
|
||||
{% else %}
|
||||
<a class="sidebar-nav-item" href="{{ link }}">
|
||||
<i class="fab fa-{{ name }}"></i>
|
||||
<i class="fa fa-{{ name }}"></i>
|
||||
</a>
|
||||
{% endif %}
|
||||
{% endfor %}
|
||||
<a class="sidebar-nav-item" href="{{ FEED_DOMAIN }}/{{ FEED_ALL_ATOM if FEED_ALL_ATOM else FEED_ALL_RSS }}">
|
||||
<i class="fas fa-rss"></i>
|
||||
<i class="fa fa-rss"></i>
|
||||
</a>
|
||||
</nav>
|
||||
</div>
|
||||
|
||||
Reference in New Issue
Block a user