Property with nullish value in inline style should be omitted

[MarvinXu]

Astro Info

Astro                    v3.4.0
Node                     v18.18.0
System                   Linux (x64)
Package Manager          npm
Output                   static
Adapter                  none
Integrations             none

If this issue only occurs in one browser, which browser is a problem?

No response

Describe the Bug

In JSX, <div style={{ backgroundColor: condition ? '#d6f9dd' : undefined }} /> will not produce any style. But is astro template, this will produce style="background-color:undefined"

What's the expected result?

behave as convention in JSX

Link to Minimal Reproducible Example

https://stackblitz.com/edit/github-47zegu?file=src%2Fpages%2Findex.astro

Participation

• I am willing to submit a pull request for this issue.

[matthewp]

This is probably a pretty simple thing to fix once you find the code where this happens (I don't remember off-hand).

[MarvinXu]

I'll fix this by the end of the week. It would be great if anyone remember where the code is and inform me 😄