search

NeteaseYanxuan / OSSA

Taro官方推荐的企业级多端 UI 库。目前已支持React语法。
https://ossa.miaode.com
MIT License
389 stars 33 forks source link

deprecatedProp 合并错误 #123

Closed CaedmonW closed 1 year ago

CaedmonW commented 1 year ago

https://github.com/NeteaseYanxuan/OSSA/blob/6573fb0252474410726b44fd09875d8ccbf38414/packages/ossa/src/utils/index.ts#L29

newProp 有可能会有默认值，导致其值不为null或undefined，导致合并的值有误。 比如Modal组件的closeable和showCloseIcon属性的默认值都是true，如果只指定showCloseIcon为false，合并后仍为true

kiner-tang commented 1 year ago

这种情况，如果新值有默认值的话，好像怎么处理都是不对的，就程序而言，有默认值就跟用户传了新值一样了。我们是不是把默认值的处理逻辑改一下，不再Index.defaultProps中定义，而是在调用deprecatedProp时传入默认值