-
-
Notifications
You must be signed in to change notification settings - Fork 127
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
Fix legacy Font Awesome icon classes #1255
Conversation
✅ Deploy Preview for pydis-static ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I noticed there are a few more FA icons on our site that needs fixing, from looking at a deploy preview from before #1232 the issue might be from upgrading FA, from which incompatible classes we might've missed. I think I'll just fix those as well in this PR together since I found them already. |
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.
It's kind of annoying that the "lazy load" animation on the timeline doesn't work properly if you try to search for anything. Oh well. Thank you for the fixes.
There's a one-liner fix for that which just changes hidden items from |
I'm bad at requesting reviews, apologies |
We were using some Font Awesome 4 classes, eg:
Which have since been removed. They don't show up correctly after switching to Font Awesome 6 (in #1232).
This is because the Kit we were previously using included some JS files that handled the backwards compatibility. (Thanks to @lemonsaurus for finding this out)
This fix replaces these font awesome classes to the most sensible replacements I could find, which are the new class names and should all display correctly.
Timeline
https://pythondiscord.com/timeline/
https://deploy-preview-1255--pydis-static.netlify.app/timeline/
1
2
Resources
https://pythondiscord.com/resources/?type=project-ideas
https://deploy-preview-1255--pydis-static.netlify.app/resources/?type=project-ideas
Code Jam results
https://www.pythondiscord.com/events/code-jams/
https://deploy-preview-1255--pydis-static.netlify.app/events/code-jams/
For CJ10, 9, 8.