Don't overwrite old styles when updating nodes with inline styles.

The current strategy for applying inline styles is to merge the old node styles with the new node styles, producing a new set of styles that has all the keys, iterate over them, apply the new styles and erase the old styles.

Old Styles

{
 color: red;
 fontSize: 14;
}

New Styles

{
 color: blue;
}

Result

{
 color: blue;
}

Should we change this behavior and instead produce the following result:

Proposed Result

{
 color: blue;
 fontSize: 14;
}

/cc @mindplay-dk @zaceno

[SkaterDad]

In your proposed output, the font size is still being set on the DOM node even though the property wasn’t present in the “new styles” object. At least that’s how I interpret your post. That seems strange, and I called it orphaned because the vdom didn’t want it there and presumably isn’t controlling it.

Am I reading this wrong?

[jorgebucaran]

@SkaterDad Correct, but it’s not being set, it’s what was already in the element. Thanks for the feedback, I’m calling this one off! 💯