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

Performance issues in processing content with multiple paragraphs containing multiple inline widgets #248

Open
mabryl opened this issue Apr 24, 2023 · 2 comments
Labels
support:2 An issue reported by a commercially licensed client. type:improvement type:performance

Comments

@mabryl
Copy link

mabryl commented Apr 24, 2023

This issue is related to this one that was reported previously.

Unfortunately, when the content is more complex than the one used in the example in the issue linked above, performance still suffers, even when two-way data binding is disabled.

The increased complexity of the content refers to the fact that now we're dealing with multiple paragraphs containing multiple inline widgets instead of a one long paragraph with numerous widgets.

A vanillaJS implementation of the editor seems to handle this type of content better than our Vue implementation.

In the attachments you can see the two implementations and compare their performance.

To open the vanilla project, open the index.html file.

To open the Vue project, install deps yarn install, and spin up the dev server yarn serve.

vanilla.zip
vue.zip

@mabryl mabryl added type:improvement type:performance squad:platform Issue to be handled by the Platform team. labels Apr 24, 2023
@pomek
Copy link
Member

pomek commented Jul 31, 2023

Internal notes

  • Moving to Needs verification because it might be already solved by disabling 2way data binding.
  • Please, check if passing the disableTwoWayBinding=true option helps.

@minyoung90
Copy link

Is there any progress at all?

@pomek pomek removed the squad:platform Issue to be handled by the Platform team. label Mar 5, 2024
@mabryl mabryl added the support:2 An issue reported by a commercially licensed client. label May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
support:2 An issue reported by a commercially licensed client. type:improvement type:performance
Projects
None yet
Development

No branches or pull requests

3 participants