salt-ui / saltui

MIT License
238 stars 64 forks source link

CheckboxField组件选择数据后,在props.data未发生变更时,无法正常显示选中的值 #231

Closed air-eienniutau closed 6 years ago

air-eienniutau commented 6 years ago

请按照以下模板填写

这是一个

SaltUI 版本

~3.10.0

浏览器、操作系统等环境信息

chrome可复现

复现步骤

~3.10.0版本会下出现CheckboxField组件slot mode下选择数据后组件无展示的bug。 fix issue #230 的修改中将 this.state.selectedText = selectedText; 修改为了 return selectedText; 但在 组件的handleConfirm中未将接受到的值setState https://github.com/salt-ui/saltui/blob/master/src/CheckboxField/CheckboxField.jsx#L123

当前demo中的CheckboxFieldDemo不会出现上述问题是因为demo中正好将onchange接受到的data再次作为props.data传递给组件,此时componentWillRecieveProps中修改了state.selectedText 而下述demo则会出现此bug https://codepen.io/air_wataru/pen/BVgqBX

eternalsky commented 6 years ago

fixed in https://github.com/salt-ui/saltui/commit/6205277a2d1ec13351c6bb26ed62db5a9f3fca93