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.

Sign up for GitHub

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

Jump to bottom

Electromagnet can be moved off-axis when "Lock to Axis" is checked. #169

Closed
pixelzoom opened this issue May 22, 2024 · 4 comments
Closed

Electromagnet can be moved off-axis when "Lock to Axis" is checked. #169

pixelzoom opened this issue May 22, 2024 · 4 comments
Assignees
@pixelzoom
Labels
type:bug Something isn't working

Comments

@pixelzoom
Copy link
Contributor

I've seen this several times, but I have not been able to identify steps to reproduce. I don't know if it also occurs with the bar magnet.
@pixelzoom pixelzoom added the type:bug Something isn't working label May 22, 2024
@pixelzoom pixelzoom self-assigned this May 22, 2024
@pixelzoom
Copy link
Contributor Author

pixelzoom commented May 22, 2024 •
edited
Loading

The background layer of the electromagnet coil is not locked to the axis. Grab and drag the background layer to reproduce.

Not a problem with the bar magnet, since it has no coil and therefore no background layer.

pixelzoom added a commit that referenced this issue May 22, 2024
@pixelzoom
rename isLockedToAxisProperty to lockToAxisProperty, #169
67cce12
@pixelzoom
Copy link
Contributor Author

In the above commits, I renamed isLockedToAxisProperty to lockToAxisProperty. That's the name I keep searching for, because it's the label on the associated checkbox, and I suspect others will have the same problem.

@pixelzoom
Copy link
Contributor Author

It's also possible to drag the electromagnet behind the right panels, by dragging it by its backgroundNode So the backgroundNode is apparently not being made aware of the drag bounds.

pixelzoom added a commit that referenced this issue May 22, 2024
@pixelzoom
fix dragBoundsProperty and cursor for electromagnet.coil.backgroundNo…
a5f6521 
…de, #169
pixelzoom added a commit that referenced this issue May 22, 2024
@pixelzoom
make ElectromagnetNode and PickupCoilNode responsible for their own c…
7138564 
…ursors, #169
@pixelzoom
Copy link
Contributor Author

Fixed in the above commits. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pixelzoom pixelzoom

Labels
type:bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pixelzoom