Skip to content
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

Sass mixins include multiple axes files #738

Open
jwr1 opened this issue May 28, 2023 · 0 comments
Open

Sass mixins include multiple axes files #738

jwr1 opened this issue May 28, 2023 · 0 comments
Labels
bug Something isn't working contributors welcome! Open to outside contributors sass Sass related issue

Comments

@jwr1
Copy link
Member

jwr1 commented May 28, 2023

When using the new sass mixins with variable fonts, the inclusion of multiple axes is not handled correctly. Instead of using the file that best matches the selected axes (I.e. if wght, ital and another are selected, standard should be used), multiple wght+axis files are included. Also, using all as the axes argument currently always resolves to full, but not all fonts have full, they might only have standard.

@jwr1 jwr1 added bug Something isn't working sass Sass related issue labels May 28, 2023
@ayuhito ayuhito added the contributors welcome! Open to outside contributors label Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working contributors welcome! Open to outside contributors sass Sass related issue
Projects
None yet
Development

No branches or pull requests

2 participants