formilyjs / antd

Formily Ant Design Adaptor
MIT License
51 stars 45 forks source link

fix: fix race condition which cause scrolling issue #14

Closed csc-bo closed 1 year ago

csc-bo commented 1 year ago

Before submitting a pull request, please make sure the following is done...

Please do not delete the above content


What have you changed?

Issue: When a user drags an item outside the visible area with autoScroll enabled, the sort handle causes the page to scroll to the top after the user stops the action.

Result: I fixed the race condition between the field.move function and any animations or updates that may be happening concurrently, which prevented the issue from persisting.

netlify[bot] commented 1 year ago

Deploy Preview for formily-antd5 ready!

Name Link
Latest commit b8c0c498e719fd71123e1b4fd4f8954e8e1f904c
Latest deploy log https://app.netlify.com/sites/formily-antd5/deploys/64719f376895ff000861f524
Deploy Preview https://deploy-preview-14--formily-antd5.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

yiyunwan commented 1 year ago

改成@formily/share里的全局适配吧,别直接使用window