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

Interaction handler keyboard events are registered twice #3536

Closed
2 tasks done
klakhov opened this issue Aug 10, 2021 · 0 comments · Fixed by #3881
Closed
2 tasks done

Interaction handler keyboard events are registered twice #3536

klakhov opened this issue Aug 10, 2021 · 0 comments · Fixed by #3881
Assignees
Labels
bug Something isn't working

Comments

@klakhov
Copy link
Contributor

klakhov commented Aug 10, 2021

My actions before raising this issue

Expected Behaviour

Interaction handler event listeners should be registered only once

Current Behaviour

Currently keyup and keydown listeners are registered twice

Possible Solution

Remove event listeners when interactionHandler class is deleted

Steps to Reproduce (for bugs)

  1. Set up breakpoint on window.addEventListener('keyup') in interactionHandler constructor
  2. Notice that application stops twice
    2021-08-10 10-47-10 (3)

Context

Your Environment

  • Git hash commit (git log -1): f2c9dc0
  • Docker version docker version (e.g. Docker 17.0.05): 20.10.8
  • Are you using Docker Swarm or Kubernetes? No
  • Operating System and version (e.g. Linux, Windows, MacOS): Linux
  • Code example or link to GitHub repo or gist to reproduce problem:
  • Other diagnostic information / logs:
    Logs from `cvat` container

Next steps

You may join our Gitter channel for community support.

@bsekachev bsekachev added the bug Something isn't working label Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants