AndyJin comments

Results 28 comments of


                                            AndyJin

antd 高级组件物料>CRUD表格物料重新选择组件分页配置会重新打开bug

目前采用这种bool|object这种联合类型的组件设置器都出问题了，应该是设置器这部分有过重构导致不兼容了。这个问题影响是非常大的，需要这部分的专家分析下该怎么处理，最好能在引擎部分继续保持兼容。 @JackLian @LeoYuan

feat: material spec add ignoreDefaultValue field to component property description

这个方案目前看可以解决问题（当然物料有一波升级，但兼容上还是比较友好）。有一个隐患就是，我当时pr里的方案增加visible判断，并且放在`this.value !== undefined`之前，主要原因是避免调用this.value。假设pagination为false时，调用 pagination.xxx 属性的设置器的 `this.value` 会导致pagination被赋值为 `{}`，因为调用this.value会创建Prop，会导致pagination的Prop的type被设置为map 如下图，我调试了Table，当pagination为false时，初始化pagination.showSizeChanger的默认值，会调用如下路径，最终会导致。我的担心是如果后续的持续迭代导致在其他环节调用了 pagination.showSizeChanger 设置器的this.value，因为会导致pagination被重置，那么后续会出现奇怪的bug，例如下图，我调整了这个代码顺序，会导致这种对this.value调用有严格的顺序要求，我觉得是个较大的隐患。