Fix: when 'value' is an empty and has 'defaultValue', 'value' will be replace.

[zhang740]

No description provided.

[developit]

What if we used this check instead of all three?

if (attrs.value==null) {
  attrs.value = attrs.defaultValue;
}

[paranoidjk]

IMO, only undefined should be ignored.

if (attrs.defaultValue) {
	if (attrs.value === undefined) {
		attrs.value = attrs.defaultValue;
	}
	delete attrs.defaultValue;
}

[paranoidjk]

actually, in check will be most strictly, but, preact-compat will merge defaultValue to value.

[developit]

What is React's behavior here? They usually differentiate between null and undefined, but I haven't tested this specific behavior. We should just use whatever they use to avoid an inconsistency.

[felixjung]

I also ran into problems with the preact-compat implementation of defaultValue.

In React, the defaultValue prop will not set the value prop on an element in general. In my observations, the value prop remains undefined. Changing the defaultValue prop does not cause any further changes/renders on the component.

[felixjung]

The described behavior applies to the case of a select.