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

Adding dynamic fields should not rerender the entire form #1507

Open
1 task
arafatahmedtanimcsedu57 opened this issue Dec 26, 2022 · 3 comments
Open
1 task
Labels

Comments

@arafatahmedtanimcsedu57
Copy link

arafatahmedtanimcsedu57 commented Dec 26, 2022

Environment

Bowser Crush

  • Hosting type
    • Local deployment
      • Version:
  • Formio.js version: 4.14.8
  • Frontend framework: ReactJS
  • Browser: Chrome
  • Browser version: latest

Steps to Reproduce

  1. Create a datagrid with 15 input field
  2. To add a dynamic row, Press Add another button, and
  3. Continue it 100 times.

Expected behaviour

  • After adding a new dynamic row, the whole form shouldn't re-render.
  • The browser shouldn't crush for any number of dynamic fields.

Observed behaviour

  • Each time the whole form is re-rendering after a new dynamic field is added.
  • After adding 100 dynamic rows Browser is crushed.

@nazmul-prince
Copy link

I'm also facing the same issue. Our client wanting to add a dynamic row consists of 7 columns for thousands of time. He cant do any further actions after adding more than 100 dynamic rows, the browser either crashed or become irresponsive.

@brendanbond
Copy link
Contributor

brendanbond commented Apr 27, 2023

@arafatahmedtanimcsedu57 thanks for filing this issue. I've reported this in our internal issue tracker and I'll try to follow up if it gets resourced.

@brendanbond brendanbond changed the title [BUG] Adding dynamic fields should not rerender the entire form Apr 27, 2023
@VikkiAlenn
Copy link

@arafatahmedtanimcsedu57 We are not able to reproduce this issue on the latest versions of formioJS (4.18.2, 4.19.0, etc.). Please, try on the latest versions and let us know if it is still the issue for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants