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

menu opens when knob drag/drop ends over empty space #72

Closed
khimaros opened this issue Mar 10, 2022 · 5 comments
Closed

menu opens when knob drag/drop ends over empty space #72

khimaros opened this issue Mar 10, 2022 · 5 comments
Labels
help wanted Extra attention is needed

Comments

@khimaros
Copy link

reproduction steps:

  1. click mouse + hold on a knob
  2. drag the cursor onto an empty space of the canvas
  3. release mouse button

what happens? menu opens

what should happen? nothing

@maximecb
Copy link
Owner

Hi @khimaros. Are you running Safari? If so, there's a bug in the Safari pointer capture API. If not, can you specify which browser and OS you're running?

@khimaros
Copy link
Author

@maximecb

Name: Firefox
Version: 96.0.3
Build ID: 20220126154723
Distribution ID:
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:96.0) Gecko/20100101 Firefox/96.0
OS: Linux 5.16.0-1-amd64 #1 SMP PREEMPT Debian 5.16.7-2 (2022-02-09)
OS Theme: Adwaita-dark / Adwaita
Multiprocess Windows: 6/6
Fission Windows: 6/6 Enabled by phased rollout
Enterprise Policies: Inactive
Google Location Service Key: Found
Google Safebrowsing Key: Found
Mozilla Location Service Key: Found
Safe Mode: false

@maximecb
Copy link
Owner

Hm. That's disappointing. It used to work in Firefox. They must have changed the behavior of the pointer capture API.

Any chance you could help me investigate? The code that handles this is here: https://github.com/maximecb/noisecraft/blob/main/public/knob.js#L41

@maximecb maximecb added the help wanted Extra attention is needed label Mar 11, 2022
@maximecb
Copy link
Owner

I implemented a hack which I think should fix the problem. Let me know if it does/doesn't :)

@khimaros
Copy link
Author

that seems to have fixed it. thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants