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

Console errors when dragging Custom block to bottom with alt input #403

Open
Nancy-Salpepi opened this issue Sep 14, 2024 · 4 comments
Open
Labels
dev:alt-input dev:help-wanted Extra attention is needed type:bug Something isn't working

Comments

@Nancy-Salpepi
Copy link

Test device
MacBook Air M1 chip

Operating System
14.6.1

Browser
Chrome

Problem description
For phetsims/qa#1141, in Buoyancy Basics on the Explore Screen, dragging a block at min density and min volume to the bottom of the pool causes an error which makes the block temporarily unmovable with alt input. Tabbing away and then back again fixes the problem.

Steps to reproduce

  1. In Basics on the Explore Screen, Select "Custom"
  2. Move the Density and Volume sliders to minimum values
  3. Tab to and grab the block
  4. Drag the block to the bottom of the pool
  5. Once the block pops back up, try to move it with alt input

Visuals

minDensity.mp4
Screenshot 2024-09-14 at 4 24 14 PM
@Nancy-Salpepi Nancy-Salpepi added the type:bug Something isn't working label Sep 14, 2024
@Nancy-Salpepi
Copy link
Author

When the block is at the minimum density but not at minimum volume (ex. 3.5 or 4.5), the block will continuously sputter when dragged to the bottom of the pool with alt input.

minDensity2.mp4

@Nancy-Salpepi
Copy link
Author

Also, when using the pointer, I can drag the block through the scale when it has the minimum density and volume values.

blockThroughScale.mp4

@Nancy-Salpepi
Copy link
Author

Following the steps in #403 (comment) in Studio shows this:
Screenshot 2024-09-19 at 9 00 30 AM

@zepumph zepumph added dev:help-wanted Extra attention is needed dev:alt-input labels Oct 1, 2024
@zepumph
Copy link
Member

zepumph commented Oct 1, 2024

Maybe related to other work we were doing with the pivot/constraint logic in #367, but I thought we reverted that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev:alt-input dev:help-wanted Extra attention is needed type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants