Skip to content

Conversation

@johnf
Copy link
Collaborator

@johnf johnf commented Oct 29, 2025

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

johnf added 4 commits October 26, 2025 12:54
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
@johnf johnf requested a review from vonovak October 29, 2025 06:55
@johnf
Copy link
Collaborator Author

johnf commented Oct 29, 2025

@vonovak @oblador can you give this a quick once over before I merge?

@johnf johnf mentioned this pull request Oct 29, 2025
@oblador
Copy link
Owner

oblador commented Oct 29, 2025

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)) {
Copy link
Collaborator

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?

Copy link
Collaborator Author

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

Copy link
Collaborator

@vonovak vonovak left a 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 :)

@johnf johnf merged commit 6292c0e into master Nov 1, 2025
16 checks passed
@johnf johnf deleted the fontawesome-7 branch November 1, 2025 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants