fix(preset-web-fonts): sort weights as string using localeCompare

[arunanshub]

Previously, the weights were converted to Number before sorting. This would lead to conversion of string to NaN.

[netlify]

✅ Deploy Preview for unocss ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	b1e08fa
🔍 Latest deploy log	https://app.netlify.com/sites/unocss/deploys/64b26be3481b1500082a3c84
😎 Deploy Preview	https://deploy-preview-2845--unocss.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[zyyv]

@arunanshub Is there any downside to using map(Number).sort((a,b)=>a-b), We are missing a step to judge whether it is a number.

[zyyv]

@arunanshub I changed the code a bit, feel free to optimize it. By the way can you add a test case for bunny. Need your review.

[arunanshub]

@zyyv In UnoCSS docs for Web fonts preset we should ideally be able to selectively choose the font weights along with their italicized versions. This is currently possible using this configuration:

{
  // ...
  fonts: {
    sans: 'Roboto',
    mono: ['Fira Mono:400,700i'],
  }
}

However, this method is sub-optimal. It is cleaner if we specify the weights like this:

{
  // ...
  fonts: {
    sans: 'Roboto',
    mono: {
      name: 'Fira Code',
      weights: [400, '700i'],
    },
  }
}

However, the current weight sorting function converts all the weight values to numbers, which converts the italicized weights to NaN.

[zyyv]

I see, I will revert my changes. Thank you for your guide.

[arunanshub]

@arunanshub I changed the code a bit, feel free to optimize it. By the way can you add a test case for bunny. Need your review.

@zyyv The removal of italicized font weights undermines the objectives of the pull request. Our intention is to sort all font weights, including the italicized versions.

[zyyv]

I didn't know the weight would carry the italics, thanks again.