Unclear what CSS values are valid for `border` when `strictPropertyValues` is enabled about panda HOT 5 CLOSED

you're right, we made a mistake by including those properties
strictPropertyValues should restrict any CSS property that has a syntax with predefined values to those values only

border can be complex like 1px solid blue so it shouldn't be restricted
borderStyle however can still be restricted

#2336

from panda.

Ah, ok, great. Thanks for the quick fix!

from panda.

@astahmer As one potential follow-up, FYI I just noticed that borderStyle is not currently in StrictableProps but has the same restricted subset of values, since it's the shorthand for the border*Style properties that are currently in StrictableProps. The one nuance is that it can have multiple values, e.g. border-style: none solid dotted dashed;.

from panda.

adding that to the list would prevent such use-case none solid dotted dashed so I guess it's better not to ?

from panda.

Ah, ok. I guess I was thinking the TS type could be something like this:

BorderStyle | `${BorderStyle} ${BorderStyle}` | `${BorderStyle} ${BorderStyle} ${BorderStyle}` | `${BorderStyle} ${BorderStyle} ${BorderStyle} ${BorderStyle}`

But maybe that's not workable for some reason? It does require a relatively new TS version to support template literal types.

from panda.