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

Migrate to Font-Awesome 5 #1438

Merged
merged 4 commits into from Apr 1, 2020
Merged

Migrate to Font-Awesome 5 #1438

merged 4 commits into from Apr 1, 2020

Conversation

stevenjoezhang
Copy link
Contributor

Issue resolved: #505

@@ -49,7 +49,7 @@
{%- if theme.post_meta.created_at %}
<span class="post-meta-item">
<span class="post-meta-item-icon">
<i class="fa fa-calendar-o"></i>
<i class="far fa-calendar"></i>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

far?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some far are free, and some of the original v4 icons are far in v5

@@ -1,4 +1,4 @@
{%- set ccIcon = '<i class="fa fa-fw fa-creative-commons"></i>' %}
{%- set ccIcon = '<i class="fab fa-fw fa-creative-commons"></i>' %}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just fused about fa/far/fab? a huge update. thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not find the official documentation for migration from v4.7 to v5. In fact, this PR is based on this gist: https://github.com/estebandelaf/scripts/blob/master/fontawesome4to5.sh

About new class names:
fas for solid
far for regular
fab for brand

@stevenjoezhang
Copy link
Contributor Author

stevenjoezhang commented Mar 30, 2020

Shall we merge this PR to v7.8.0 release (April 1st)?

@stevenjoezhang stevenjoezhang added this to the 7.8.0 milestone Mar 30, 2020
@stevenjoezhang stevenjoezhang modified the milestones: 7.8.0, 7.9.0 Apr 1, 2020
@stevenjoezhang stevenjoezhang merged commit e61c2e0 into master Apr 1, 2020
Guoye added a commit to Guoye/hexo-theme-next that referenced this pull request Apr 22, 2020
* 'master' of https://github.com/theme-next/hexo-theme-next: (99 commits)
  Optimize main-menu & sidebar-nav style
  Optimize dark mode blockquote style
  Use fixed-width icons in copy-btn
  Fix exturl title
  Fix Font Awesome path and font-family
  Fix Font Awesome class name
  Migrate to Font Awesome 5 (theme-next#1438)
  Release v7.8.0
  Optimize dark mode button color
  Fix theme-next#1424: Dark mode footer color of scheme Mist
  Optimize dark mode colors
  Combine .content and .posts-expand
  Optimize mobile_layout_economy
  New Crowdin translations (theme-next#1413)
  Some minor fixes
  Revert theme-next#1124: Class name for pages
  Run lebab
  Trigger search after loading is complete
  Simplify footer.powered settings (theme-next#1432)
  Remove `seo` option in `_config.yml` (theme-next#1414)
  ...

# Conflicts:
#	_config.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants