ant-design / ant-design-pro-layout

🌃 Powerful and easy to use beautiful layout
https://prolayout.ant.design/
MIT License
1k stars 238 forks source link

👑 [需求] WrapContent.tsx 文件引入的ConfigProvider配置有可能覆盖项目自定义的配置 #552

Closed aisriver closed 4 years ago

aisriver commented 4 years ago

🥰 需求描述 [详细地描述需求,让大家都能理解]

比如我在项目中使用 ` <ConfigProvider getPopupContainer={trigger => trigger.parentElement || document.body}>

` getPopupContainer 这个属性会被ProLayout里面的覆盖,导致弹窗里面的Select选项浮动出现问题

🧐 解决方案 [如果你有解决方案,在这里清晰地阐述]

1、这样写能解决问题,但其他用户可能不容易排查到

`

trigger.parentElement || document.body}>{props.children}

`

2、优化WrapContent文件的写法,getPopupContainer以用户的配置优先

🚑 其他信息 [如截图等其他信息可以贴在这里]

image

chenshuai2144 commented 4 years ago

看一下,我应该移除掉这个东西

aisriver commented 4 years ago

看一下,我应该移除掉这个东西

OK!