Closed oasis-cloud closed 1 month ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 83.88%. Comparing base (
7b1ed92
) to head (3902ac1
). Report is 1 commits behind head on next.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
此拉取请求引入了一个新的实用函数mergeProps
,用于将defaultDialogProps
与传入的props
合并,修改了DialogWrap
功能组件中属性的解构方式。直接解构props
的方式被替换为一个包含默认值的合并对象。DialogWrap
的defaultProps
赋值已被移除,默认值现在通过mergeProps
函数集成。解构的属性现在包括closeOnOverlayClick
和lockScroll
,这些属性预计将成为合并属性的一部分。DialogWrap
组件的整体结构和功能保持不变,但默认属性的处理得到了简化。
文件路径 | 更改摘要 |
---|---|
src/packages/dialog/dialogwrap.tsx | 引入mergeProps 函数,修改属性解构方式,移除defaultProps 。 |
src/utils/merge-props.ts | 添加mergeProps 方法。 |
defaultProps
的移除,类似于主PR中对DialogWrap
组件的更改。Dialog
组件相关,直接与主PR中的DialogWrap
组件相连,涉及属性处理的修改。🐇 在月光下轻轻跳跃,
新的功能如花绽放,
合并属性更简单,
对话框更聪明,
兔子欢快歌唱,
代码如梦般流畅。 🌙✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
🤔 这个变动的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
☑️ 请求合并前的自查清单
⚠️ 请自检并全部勾选全部选项。⚠️
Summary by CodeRabbit
DialogWrap
组件中引入了新的属性合并功能,优化了默认属性的处理方式。DialogWrap
组件的属性声明,确保新功能的清晰性。