solidjs / solid / 34729625842

72%

main: 89%

push

github

fix(web): omit nullish style/class in ssrElement, no stray tag whitespace (#3382) (#3393)

`ssrElement` tested `style` and `class` before the nullish early-out, so a
spread whose style/class resolved to undefined serialized `style=""
class=""` while every other attribute was omitted — and while the client
removes the attribute for the same value. Move the two branches below the
early-out.

Emitted attributes now carry their own leading space (as the hydration key
already does) instead of a trailing one appended per key, so skipped props
leave nothing behind: `<li _hk=0>` rather than `<li _hk=0 >`. Void elements
keep the space before `/>` because the hydration key is unquoted.

Co-authored-by: Cursor <cursoragent@cursor.com>

519 of 790 branches covered (65.7%)

Branch coverage included in aggregate %.

772 of 1007 relevant lines covered (76.66%)

15.01 hits per line