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

About Transfer component performance issues #9703

Closed
hqwlkj opened this issue Mar 16, 2018 · 6 comments
Closed

About Transfer component performance issues #9703

hqwlkj opened this issue Mar 16, 2018 · 6 comments

Comments

@hqwlkj
Copy link
Contributor

hqwlkj commented Mar 16, 2018

Version

3.3.0

Environment

Operating system: macOS high Sierra, browser: Safari browser, version 11.0.3 (13604.5.6), react:^16.2.0

Reproduction link

https://ant.design/components/transfer/

Steps to reproduce

When the amount of data is greater than 2,000, the UI rendering is slow and very fast.

What is expected?

It is recommended to optimize

What is actually happening?

@ant-design-bot
Copy link
Contributor

Hello @hqwlkj. Please provide a online reproduction by forking this link https://u.ant.design/codesandbox-repro. Issues labeled by Need Reproduce will be closed if no activities in 7 days.

@hqwlkj
Copy link
Contributor Author

hqwlkj commented Mar 16, 2018

Hello @afc163 Please check this link https://codesandbox.io/s/rjvv4qk6yn,

Large amount of data, the operation is very cautious, it is recommended to optimize.

@yesmeck yesmeck removed the 🤔 Need Reproduce We cannot reproduce your problem label Mar 23, 2018
@hqwlkj
Copy link
Contributor Author

hqwlkj commented Mar 28, 2018

@yesmeck https://codesandbox.io/s/rjvv4qk6yn, Reproduce address

@liangchaofei
Copy link

数据大,还是会卡,复现demo:https://codesandbox.io/s/upbeat-brown-5qkm9

@aditya-padhi-kbl
Copy link
Contributor

Hi @afc163 Thank-you for pointing to this issue in #30158
May I know if there are any plans to fix this perf issue as it will be very helpful to a lot of users.
I feel it will be great if we could support virtualization natively in Transfer component.
Here is a sample implementation I did by using react-window.
https://stackblitz.com/edit/react-ofcadv-gcds33?file=index.js

You have to go to line number 129 and set perfMode to true.

However, I am not able to setup some of the default behavior of transfer component like selectAll, individual item select etc.

Once again I would like to thank-you for your quick guidance.

Kind Regards,
Aditya

@afc163
Copy link
Member

afc163 commented Jan 11, 2023

Transfer perfermance has been improved in #39465, also you can try this method of #40040 .

See https://codesandbox.io/s/pensive-kate-8nsiu1?file=/index.js

@afc163 afc163 closed this as completed Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants