Repository navigation
Incorrect font-feature-settings on multiple values for font-variant-numeric #1546
Description
Bug description
When I specify multiple values for font-variant-numeric, PostCSS just copies them in font-feature-settings which makes feature declarations invalid.
Source CSS
.lining-nums {
font-variant-numeric: lining-nums;
}
.tabular-nums {
font-variant-numeric: tabular-nums;
}
.tabular-and-lining-nums {
font-variant-numeric: tabular-nums lining-nums;
}Expected CSS
.lining-nums {
font-feature-settings: "lnum";
font-variant-numeric: lining-nums;
}
.tabular-nums {
font-feature-settings: "tnum";
font-variant-numeric: tabular-nums;
}
.tabular-and-lining-nums {
font-feature-settings: "tnum", "lnum";
font-variant-numeric: tabular-nums lining-nums;
}Actual CSS
.lining-nums {
font-feature-settings: "lnum";
font-variant-numeric: lining-nums;
}
.tabular-nums {
font-feature-settings: "tnum";
font-variant-numeric: tabular-nums;
}
.tabular-and-lining-nums {
font-feature-settings: tabular-nums lining-nums;
font-variant-numeric: tabular-nums lining-nums;
}Playgound example
Does it happen with npx @csstools/csstools-cli <plugin-name> minimal-example.css?
N/A
Debug output
No response
Extra config
No response
What plugin are you experiencing this issue on?
PostCSS Preset Env
Plugin version
10.1.3
What OS are you experiencing this on?
macOS
Node Version
23.5.0
Validations
- Follow our Code of Conduct
- Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
Would you like to open a PR for this bug?
- I'm willing to open a PR
Activity
romainmenke commentedon Jan 5, 2025
Hi @naiyerasif
I think this is the same issue as postcss/postcss-font-variant#16
postcss-preset-envcontains that plugin.I will take a closer look at this feature and see if we can improve things.