Skip to content
This repository was archived by the owner on Jan 6, 2023. It is now read-only.

Investigate performance syncing long text #783

Closed
vladikoff opened this issue Mar 20, 2018 · 1 comment
Closed

Investigate performance syncing long text #783

vladikoff opened this issue Mar 20, 2018 · 1 comment
Assignees
@sebastienbarbier
Milestone
Triaged

Comments

@vladikoff
Copy link
Contributor

[Description]:

Might be able to use a WebWorker or some other cool web tech (tm)

Extracted from: #743
@vladikoff vladikoff added this to the Triaged milestone Mar 20, 2018
@sebastienbarbier sebastienbarbier mentioned this issue Mar 20, 2018
Merged
16 tasks
sebastienbarbier added a commit that referenced this issue Mar 26, 2018
@sebastienbarbier
Delay editor on change event since copy pasting was trigger 100 of it.
d227670 
Related to #783
This was referenced Mar 28, 2018
Closed
Closed
@sebastienbarbier
Copy link
Collaborator

I believe this issue was actually a mix of few issues noew fixed and should be closed.

Server was never deleting notes so syncing was way longer that it should be. Fixed in #824.
Copy/pasting was taking ages because of unoptimized behaviour in ckeditor. Fixed in #839.

We can still improve performances using webworker on background side, but this involve important refactoring which will be done in future release. An issue has been open : #822

Based on those, I will close this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sebastienbarbier sebastienbarbier

Labels
None yet
Projects
None yet
Milestone
Triaged
Development

Successfully merging a pull request may close this issue.

Fix copy/paste delay in editor and build backgound.js using webpack mozilla/notes
2 participants
@vladikoff @sebastienbarbier