Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Popover: Position is incorrect when content size has changed #14977

Closed
1 task done
AshoneA opened this issue Feb 21, 2019 · 4 comments
Closed
1 task done

Popover: Position is incorrect when content size has changed #14977

AshoneA opened this issue Feb 21, 2019 · 4 comments
Assignees
Labels
🐛 Bug Ant Design Team had proved that this is a bug. 🙅🏻‍♀️ WON'T RESOLVE

Comments

@AshoneA
Copy link
Contributor

AshoneA commented Feb 21, 2019

  • I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

https://ant.design/components/popover-cn/

Steps to reproduce

https://codepen.io/AshoneA/pen/yZWyGK

What is expected?

It should still has correct postion when content size has changed

What is actually happening?

Is it possible to get correct position?

Environment Info
antd 3.13.5
React 16
System MacOS
Browser Chrome
@ztplz
Copy link
Contributor

ztplz commented Feb 21, 2019

#14582

@afc163 afc163 added the 🐛 Bug Ant Design Team had proved that this is a bug. label Feb 21, 2019
@zombieJ
Copy link
Member

zombieJ commented Feb 26, 2019

Popup don't listen content size since perf consider. You can force realign if needed: https://codesandbox.io/s/9q16jwl194

@ztplz
Copy link
Contributor

ztplz commented Feb 26, 2019

那个bug不修了吗 dom-align在四个方向匹配不成功后会重置points 但是接下来 它又会尝试把元素移动到可视区域 但是箭头是根据points来算的 我那个pr是根据两个元素的中心点来算的 只算四个方向

@zombieJ
Copy link
Member

zombieJ commented Feb 26, 2019

那个bug不修了吗 dom-align在四个方向匹配不成功后会重置points 但是接下来 它又会尝试把元素移动到可视区域 但是箭头是根据points来算的 我那个pr是根据两个元素的中心点来算的 只算四个方向

那个继续修的哈,我想要一个 example 来手工测一下。这个 issue 的问题和它不一样,因为它缺少检查时机。以前考虑过监听 size,感觉这些场景都是可以用户自己重定位的。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Ant Design Team had proved that this is a bug. 🙅🏻‍♀️ WON'T RESOLVE
Projects
None yet
Development

No branches or pull requests

5 participants