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
MAINT: update icons to fit Fontawesome 6 #963
Conversation
So far so good, I played around in the pages and all the expected icons seem to be there, let me know if you are satisfied with the changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me in general! One quick comment
@@ -75,6 +75,10 @@ function macroTemplate({ compilation }) { | |||
href="{{ pathto('_static/vendor/fontawesome/${ | |||
vendorVersions.fontAwesome | |||
}/webfonts/fa-brands-400.woff2', 1) }}"> | |||
<link rel="preload" as="font" type="font/woff2" crossorigin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How were we able to use fa
regular before if we weren't loading it with webpack? Shouldn't we also have to remove something?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's an excellent question, the only regular we were using was the moon icon of the theme switcher. the problem with fontawesome is that it's everywhere on the web so maybe we all cache it somewhere in our browser
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that seems like the most likely explanation.
@@ -75,6 +75,10 @@ function macroTemplate({ compilation }) { | |||
href="{{ pathto('_static/vendor/fontawesome/${ | |||
vendorVersions.fontAwesome | |||
}/webfonts/fa-brands-400.woff2', 1) }}"> | |||
<link rel="preload" as="font" type="font/woff2" crossorigin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that seems like the most likely explanation.
They are controlled by pydata-sphinx-theme.css. However, pydata-sphinx-theme changed the classes in v0.11.0 pydata/pydata-sphinx-theme#963. We need to update this as well.
Fix #881
fab
,fas
andfar
byfa-solid
,fa-regular
andfa-brands
Bonus: