Error parsing `font: inherit`

[nattog]

Parsing font: inherit throws an error, as no value is provided for font-size. Is this to be expected?

For context, @signal-noise/stylelint-scales uses parse-css-font, and has a workaround to bypass parse-css-font in place in case of font: inherit declarations. See issue: jeddy3/stylelint-scales#36

[jednano]

I can't say I remember what my intent was back in 2015, but my guess would be that I was assuming that because the CSS font property is already an inherited property any sort of inheriting would be redundant, right?

But I just found this SO post, which led me to creating this Codepen to verify that there is, in fact, a very good reason to support font: inherit, so thanks for pointing this out!

I believe it should not throw an error; however, I think it requires more thought behind how this affects the result object. Would it just become this?

{
	"size": "inherit",
	"family": ["inherit"],
	"style": "inherit",
	"variant": "inherit",
	"weight": "inherit",
	"stretch": "inherit",
	"lineHeight": "inherit"
}

☝️ is that how it actually works?

[jednano]

Furthermore, it's not just inherit that needs to be handled, but all other css-global-keywords as well. And from a parsing standpoint, how do I know for sure which property you are trying to inherit? This could get a bit complicated.

[finalight]

hi, may I know are you working on this issue? i also encounter this same problem

[jednano]

I'm not working on this because I don't have a strategy yet. Do you have any ideas how to solve this problem gracefully?

[finalight]

not a graceful solution, but let it pass through instead of throwing the error. Will that work?

[jednano]

Can't do that without returning an object, so the question is what does that object look like?