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 in Bounding Box Tool #8132

Closed
3 tasks
knollengewaechs opened this issue Oct 17, 2024 · 2 comments · Fixed by #8253
Closed
3 tasks

Cursor in Bounding Box Tool #8132

knollengewaechs opened this issue Oct 17, 2024 · 2 comments · Fixed by #8253
Assignees

Comments

@knollengewaechs
Copy link
Contributor

Context

In bounding box tool, the "move" cursor does not appear as expected

Expected Behavior

Within the bounding box tool, the move cursor should appear when ctrl is pressed, as moving the whole box is possible from there on

Current Behavior

If one is hovering the edges of a bounding box and then pressing ctrl, the cursor does not change. One has to move the cursor a little for it to change and take the shape of four arrows.

Steps to Reproduce the bug

  • Cannot reproduce the bug anymore / needs deeper investigation.
  1. Open annotation, activate BBox Tool
  2. Hover edge of a BBox
  3. Press Ctrl

Your Environment for bug

Firefox for Mac

  • Specific to long-running jobs (set jobsEnabled=true in application.conf)
  • Specific to webknossos.org (set isDemoInstance=true in application.conf)
@MichaelBuessemeyer
Copy link
Contributor

MichaelBuessemeyer commented Oct 17, 2024

Within the bounding box tool, the move cursor should appear when ctrl is pressed, as moving the whole box is possible from there on

If one is hovering the edges of a bounding box and then pressing ctrl, the cursor does not change. One has to move the cursor a little for it to change and take the shape of four arrows.

Yeah, I did not develop this to the fullest 😅 . #7892 (review) and #7892 (comment) are relevant here. So this is expected, although I agree that this is no ideal.

Feel free to improve the behaviour (who ever gets assigned) :)

@knollengewaechs
Copy link
Contributor Author

Good to know!

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.

2 participants