open-formulieren / open-forms

Smart and dynamic forms
https://open-forms.readthedocs.io
Other
35 stars 27 forks source link

Changing step name can cause form builder to crash #3546

Open LaurensBurger opened 11 months ago

LaurensBurger commented 11 months ago

Product versie / Product version

2.2.0 - 2.4.0

Omschrijf het probleem / Describe the bug

internal: Venr 81

When you change the name of a step, a certain amount of delay can be noticed. If you do this fast enough, the form builder will crash - which makes the entire browser tab unresponsive.

The more components in a step, the more obvious this issue becomes.

image image Memory keeps rising: image until: image

Workaround is to paste a new form name, instead of typing.

joeribekker commented 11 months ago

Refinement: This requires investigation on why this happens. Currently we have no idea.

@LaurensBurger is there any user that reported this? If you could give us a form export that would be great.

LaurensBurger commented 11 months ago

gottagofast.zip

Just change the step name, type fast. I think memory of the device is also a factor, so if you have 32gb you might not experience the crash - but you will notice the tab become sluggish and perhaps see the map component turn grey.

I can reproduce it by just pressing & holding backspace from the end of the title.

joeribekker commented 11 months ago

But, @LaurensBurger is there any client that reported this or is it a test case from you?

LaurensBurger commented 11 months ago

@joeribekker Yes Venr 81