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

refactor: Graphics Component simplification #2893

Merged
merged 2 commits into from
Jan 20, 2024

Conversation

eonarheim
Copy link
Member

@eonarheim eonarheim commented Jan 19, 2024

===:clipboard: PR Checklist :clipboard:===

  • 📌 issue exists in github for these changes
  • 🔬 existing tests still pass
  • 🙈 code conforms to the style guide
  • 📐 new tests written and passing / old tests updated with new scenario(s)
  • 📄 changelog entry added (or not needed)

==================

This PR refactors the GraphicsComponent to be simpler.

  1. Remove confusing Graphics Layering from ex.GraphicsComponent, recommend we use the ex.GraphicsGroup to manage this behavior

  2. Update ex.GraphicsGroup to be consistent and use offset instead of pos for graphics relative positioning

@eonarheim eonarheim added the api change Breaking api change label Jan 19, 2024
@eonarheim eonarheim merged commit ff73735 into main Jan 20, 2024
5 checks passed
@eonarheim eonarheim deleted the refactor/graphics-component-simplification branch January 20, 2024 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api change Breaking api change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant