-
Notifications
You must be signed in to change notification settings - Fork 70
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
Candidacy process interrupted, editor caused CPU load, transaction not picked up #3291
Comments
We are not sure what can be done here, so pls let me know if this repeats and we will re-open. @traumschule |
It needs a performance audit, for example with dev tools. |
ok re-opened, added to community pipe, pls feel free to take this one on at some point. But pls bear in mind that issues marked with label:mainnet are of highest prio. cc @singulart |
✅ Tested on https://dao-git-carthage-joystream.vercel.app/#/election While filling out the application, I tried to load the editor as much as possible and keep the modal open for a long time. Sometimes, especially when I inserted large chunks of text into the editor, the CPU load jumped up to 90 percent, but then quickly returned to normal. In general, I did not encounter the problems described above, the process of editing and submission was successful. 2022-11-02.14.26.48.mov |
not reproducible on dev, closing |
Route: https://dao.joystream.org/#/election, last step of
Announce Candidacy
modalTLDR Users wan't to use keyboard navigation and click to position the cursor without the editor changing cursor position.
Reported here and here.
The candidacy editor does some tricks to position cursor (or it may be a side-effect of
onChange
updating content), i suggest to remove that / time it better (like after 500ms user inactivity or similar). Reasons and observations:related:
pioneer/packages/ui/src/common/components/CKEditor/CKEditor.tsx
Line 134 in 825d327
The text was updated successfully, but these errors were encountered: