Skip to content

Fix the prevention of creating degenerate objects with keyboard contols. #693

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

Merged
merged 1 commit into from
Jul 6, 2022

Conversation

drgrice1
Copy link
Member

When graphing a parabola (via any of the tools that do so) or a cubic it is possible to create degenerate objects. The code that is supposed to prevent that wasn't quite working right. The jsxgraph board object needs to be updated after setting the position of a point programatically.

To test this graph any of those objects and then move one of the defining points of the object around with the keyboard. For example, graph a three point quadratic and try to move one point onto the same vertical line as another point. That should not be possible.

…ols.

When graphing a parabola (via any of the tools that do so) or a cubic it is possible to create degenerate objects.  The code that is supposed to prevent that wasn't quite working right.  The jsxgraph board object needs to be updated after setting the position of a point programatically.

To test this graph any of those objects and then move one of the defining points of the object around with the keyboard.  For example, graph a three point quadratic and try to move one point onto the same vertical line as another point.  That should not be possible.
@pstaabp
Copy link
Member

pstaabp commented Jun 29, 2022

See #676 for a demo problem.

@drgrice1
Copy link
Member Author

@pstaabp pstaabp merged commit 85bb10b into openwebwork:PG-2.17 Jul 6, 2022
@drgrice1 drgrice1 deleted the fix-keyboard-degenerates branch July 18, 2022 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants