-
Notifications
You must be signed in to change notification settings - Fork 12.2k
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
TypeScript: Some T-shirt sizes not listed in SizeProps #18788
Comments
I believe this is as simple as updating the type, so I have created this PR: #18789. |
Thanks. I've assigned both @mlwilkerson and @robmadole |
Thanks for reporting this and creating the PR, @daven97! I'm running into this issue, too. So I'll be grateful when the PR is merged. |
I can see it fixed in 6.2.0 Font-Awesome/js-packages/@fortawesome/fontawesome-svg-core/index.d.ts Lines 22 to 38 in d3a7818
|
Sorry I missed that this issue had been closed. This is working for me now in 6.2.0! Thanks! |
TypeScript error saying that the
xl
size is invalid.The docs at https://fontawesome.com/docs/web/style/size has this list of size options:
and the fontawesome-react docs at https://fontawesome.com/docs/web/use-with/react/style#size state:
However, looking at @robmadole's changes to
SizeProp
in https://github.com/FortAwesome/Font-Awesome/blob/65f4bdb5025759ba25cd3432568de288b6a67527/js-packages/%40fortawesome/fontawesome-svg-core/index.d.ts as part of #18652, several of the sizes from the table above are missing, includingxl
.The icon is displaying at the correct size, so this is simply a TypeScript error.
The text was updated successfully, but these errors were encountered: