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

Moving a Component with anything other than moveTo can lead to failed bounds check #336

Closed
adam-arold opened this issue Nov 2, 2020 · 0 comments
Closed

Moving a Component with anything other than moveTo can lead to failed bounds check #336

adam-arold opened this issue Nov 2, 2020 · 0 comments
Assignees
@adam-arold
Labels
bug Bugs within the program
Milestone
2020.2.0-RELEASE

Comments

@adam-arold
Copy link
Member

adam-arold commented Nov 2, 2020 •
edited
Loading

This happens because DefaultComponent implements Movable and delegates to an instance of Movable. because of this delegation calls to methods like moveRightBy will call the delegate's moveBy ignoring DefaultComponent's own implementation.
@adam-arold adam-arold added the bug Bugs within the program label Nov 2, 2020
@adam-arold adam-arold self-assigned this Nov 2, 2020
@adam-arold adam-arold added this to the 2020.2.0-RELEASE milestone Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adam-arold adam-arold

Labels
bug Bugs within the program
Projects
None yet
Milestone
2020.2.0-RELEASE
Development

No branches or pull requests
1 participant
@adam-arold