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

Fast Entry still works slow for longer lists of items #1096

Closed
porgabi opened this issue Mar 4, 2021 · 6 comments
Closed

Fast Entry still works slow for longer lists of items #1096

porgabi opened this issue Mar 4, 2021 · 6 comments
Labels
Milestone

Comments

@porgabi
Copy link

porgabi commented Mar 4, 2021

Are you requesting a feature, reporting a bug or asking a question?

Reporting a bug.

What is the current behavior?

When using the fast entry feature of certain widgets in the survey editor, pasting a long list of items into the Fast Entry box causes the page to become very slow and unresponsive. Clicking on elements in the editor is delayed while the text from the clipboard is being processed, though these clicks are applied once the loading is done. This can be seen towards the end of the repro video (from 01:07).
List of items example: https://pastebin.com/raw/Hy9nuyP4

Video of bug in action:

Fast.Entry.bug.mp4

What is the expected behavior?

Page and survey editor should not slow down to this extent when pasting longer text from clipboard.

How would you reproduce the current behavior (if this is a bug)?

  1. Create a dropdown question in the survey editor.
  2. Go to choices, paste above list of items into Fast Entry field.
  3. Observe editor and page slowing down drastically.

Specify your

  • Browser: Firefox, Chrome
  • Editor version: 1.8.23

Other

I am aware this bug was found and supposedly fixed in another issue (#903), though it still seems to persist. This was tested on multiple different machines so it's unlikely to be hardware related.

@andrewtelnov
Copy link
Member

@porgabi I have fixed this issue by the commit above.

Thank you,
Andrew

@porgabi
Copy link
Author

porgabi commented Mar 5, 2021

Thanks!

@porgabi
Copy link
Author

porgabi commented Mar 11, 2021

@andrewtelnov

I have updated the editor to the 1.8.34 version from NPM but the issue still appears to be there. The exact same thing happens as in the original bug report video.

I checked page source to make sure the correct survey-creator script version is used and it seems to be in order:
image

Do you have any ideas as to what could be behind this?

@andrewtelnov
Copy link
Member

@porgabi I did the following. I generate in Excel over 1000 items from item5 to item1200, go to our website, drop a dropdown question then go to choices/Fast entry and insert these items. It took about 3-4 seconds to insert them. After that everything works as expected.
The fix was is the following. If you inserting over 20 items, then we do not add all items by one with all related notication, but we reasign question.choices property with new items.

Could you please check, do you have the same issue on our website?

Thank you,
Andrew

@porgabi
Copy link
Author

porgabi commented Mar 11, 2021

It works properly on your demo site for me as well, the same issue does not come up. Which leads me to believe it might be something to do with our platform somehow. We use the creator in an OrchardCore application and haven't experienced compatibility issues of any kind so far, so this is a bit puzzling.

@andrewtelnov
Copy link
Member

@porgabi I beleive it uses a cached version in your case.

Thank you,
Andrew

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

2 participants