-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
feat: upgrade FontAwesome to version 7 with new package strucutre #1857
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
Conversation
structure Add comprehensive FontAwesome 7 support by splitting into separate packages for each style variant. This includes 3 free variants (brands, regular, solid) and 15 pro variants covering standard, duotone, and sharp styles across different weights. Key changes: - Add 18 new FontAwesome packages (3 free, 15 pro) - Update main README to list FontAwesome 7 (7.1.0) as current version with 2,806 free and 75,767 pro icons - Move FontAwesome 6 to "no longer maintained upstream" section - Add generate-fontawesome-glyphmap.mts script for generating package-specific glyphmaps from upstream CSS - Update fetch-pro.sh script with improved shell quoting - Regenerate fonts for ant-design, entypo, and evil-icons - Update directory package to clean generated files before rebuilding - Update various package READMEs with formatting improvements - Update generator template README
|
Only reviewed on my phone so didn't run test it, but LGTM! |
| } | ||
| const codePoint = Number.parseInt(content.slice(1), 16); | ||
| let codePoint = Number.parseInt(content.slice(1), 16); | ||
| if (Number.isNaN(codePoint)) { |
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.
when is this an issue?
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.
In FA7 most are what you'd expect e.g.
.fa-caravan-simple {
--fa: "\e000";
}But they have a bunch like this
.fa-bracket {
--fa: "\[";
}
.fa-bracket-left {
--fa: "\[";
}
.fa-slash-back {
--fa: "\\";
}So we need to convert the ASCII character into a code point
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.
thanks, I think this is good, just added one comment with a question :)
Add comprehensive FontAwesome 7 support by splitting into
separate packages for each style variant. This includes 3 free
variants (brands, regular, solid) and 15 pro variants covering
standard, duotone, and sharp styles across different weights.
Key changes:
version with 2,806 free and 75,767 pro icons
package-specific glyphmaps from upstream CSS
rebuilding