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

Cursor jumps while typing comments for node #7369

Closed
3 tasks
hotzenklotz opened this issue Oct 9, 2023 · 0 comments · Fixed by #7390
Closed
3 tasks

Cursor jumps while typing comments for node #7369

hotzenklotz opened this issue Oct 9, 2023 · 0 comments · Fixed by #7390

Comments

@hotzenklotz
Copy link
Member

Context

When adding a comment to a skeleton node, the typing cursor jumps around/to the front of the text input while typing. This makes it extremely hard to type meaningful comments / long texts. Likely a React issues with change events / state and constantly re-rendering.

I suspect a regression in the InputComponent. A recent change was introduced as part of the "ND"-PR:
978dde5#diff-6ad6009d91cd481c4afb4d0f83e258a9a1a61374935cfc36592454b5890d9329

image

See also Slack

Expected Behavior

I should be able to type a full sentence into the input text field without weird jumps.

Current Behavior

The text cursor in the input text field makes weird jumps while typing.

Steps to Reproduce the bug

  • Cannot reproduce the bug anymore / needs deeper investigation.
  1. Create a skeleton with at least one node
  2. Switch to Comments tab
  3. Start typing a comment. Jumpy McJump

Your Environment for bug

  • Browser name and version: Chrome 117.0.5938.149
  • Operating System and version: MacOS
  • Version of WEBKNOSSOS (Release or Commit): 24973
  • Specific to long-running jobs (set jobsEnabled=true in application.conf)
  • Specific to webknossos.org (set isDemoInstance=true in application.conf)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant