search

nuxt / fonts

Plug-and-play web font optimization and configuration for Nuxt apps.
https://nuxt.com
MIT License
370 stars 4 forks source link

feat: support variable axis #141

Open qwerzl opened 3 weeks ago

qwerzl commented 3 weeks ago

This PR adds support for variable font axis. It's quite sad that AFAIK only Google provides this level of configuration.

codecov-commenter commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 85.00000% with 12 lines in your changes are missing coverage. Please review.

Project coverage is 64.50%. Comparing base (ed5f704) to head (725d358).

Files Patch % Lines
playground/pages/providers/google.vue 0.00% 11 Missing :warning:
src/module.ts 83.33% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #141 +/- ## ========================================== + Coverage 63.70% 64.50% +0.79% ========================================== Files 41 41 Lines 2717 2789 +72 Branches 397 426 +29 ========================================== + Hits 1731 1799 +68 - Misses 960 966 +6 + Partials 26 24 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

qwerzl commented 2 weeks ago

Hi @danielroe, the type issues are fixed and I've added tests for Recursive. Sorry I didn't see the failed checks yesterday so it took some time.

qwerzl commented 2 weeks ago

@danielroe I've also thought about this, but I'm not sure whether font-variation-settings allows a range of values.