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

Add an id property to all the elements and implement new methods #198

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

elchininet
Copy link
Owner

This pull request implements a new id property in all the elements and implement new methods.

  1. Add an optional id property to all the elements. If the id is not specified as a property of an instance, a random id is generated.
  2. Implement new methods in container classes (as IsometricCanvas and IsometricGroup):
    • getChildByIndex method to return a child taking into account its index
    • getChildById method to return a child taking into account its id
    • removeChildById method to remove a child taking into account its id

@elchininet elchininet added documentation Improvements or additions to documentation enhancement New feature or request demo labels Oct 10, 2024
@coveralls
Copy link

Coverage Status

coverage: 100.0%. remained the same
when pulling 9234074 on implement_id_and_new_methods
into bac797d on master.

@elchininet elchininet merged commit 6a2e665 into master Oct 10, 2024
2 checks passed
@elchininet elchininet deleted the implement_id_and_new_methods branch October 10, 2024 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
demo documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants