fix: it was impossible to remove array or object attributes when setting the JS prop to null.

[trusktr]

• Bug
• Feature

Requirements

• Read the contribution guidelines.
• Wrote tests.
• Updated docs and upgrade instructions, if necessary.

Rationale

It is impossible for prop.serialize to return null if it simply returns JSON.stringify(val), because a string can not be coerced to null, therefore doing something like this.someProp = null does not remove the target attribute.

Implementation

Checks if val is null, and if it is just returns null.

[treshugart]

Is this still an issue in skatejs/element?

[trusktr]

Seems like it is, here:

skatejs/packages/element/src/props.ts

Line 18 in 359057e

serialize: (elem, name, oldValue, newValue) => JSON.stringify(newValue)

[treshugart]

Sorry for the late reply. This looks pretty good, but would you mind writing a test for it?