Input props remove assign

[shepherdwind]

问题

{...props} 等效于 assign({}, props)。

assign 方法在 node 和 浏览器上表现不一致，chrome 最新版本 assign 之后，安装 key 的字母顺序。这会导致浏览器和服务器渲染不一致：

浏览器上相当于

<input type="checkebox" className="foo"/>

服务端

<input className="foo" type="checkebox" />

[coveralls]

Coverage increased (+0.3%) to 84.746% when pulling e7d4cdd on input-props-sort into 31a3566 on master.

[xujihui1985]

这种透传的方式是很蛋疼，最好全都是固定下来的

[shepherdwind]

感觉这里 type 直接写死应该也没问题，checkbox 难道 type 还会是其他？style 感觉更加不需要传递了，但不知道会不会有兼容问题，就也写上了。 @yiminghe 看看？

[benjycui]

不这样传，如果用户用了 disabled 之类，要一个一个加上去么。。

[yiminghe]

disabled 处理下， style 处理倒是错误的， style 是在顶层的

[coveralls]

Coverage increased (+0.3%) to 84.746% when pulling e7c60bb on input-props-sort into 31a3566 on master.

[afc163]

facebook/react#4616

[afc163]

facebook/react#6451

[afc163]

我感觉都写死很不好，看上面的 issues 这算是老 v8 的 bug。sindresorhus/object-assign#32 针对这些问题搞了一个解决方案。

@shepherdwind 看看是否可以在服务端用 [email protected] 来解决？

应该不行，需要 babel 解决，再持续关注吧。

[yiminghe]

这个固定写死也没多少，否则以后 typescript 类型没法写

[paranoidjk]

facebook/react#6451 (comment)

react 官方引入了 object-assign ....