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

Wrap Tile over a data type #172

Merged
merged 2 commits into from
Feb 23, 2022
Merged

Wrap Tile over a data type #172

merged 2 commits into from
Feb 23, 2022

Conversation

aleokdev
Copy link
Contributor

Step for #166. Renames Tile to TileData, makes it private and uses a wrapper type over it. Makes the interface more consistent and allows adding functions that require both Tileset and Tile, such as UV calculations.

@aleokdev aleokdev added enhancement breaking This change breaks backwards-compatibility labels Feb 23, 2022
bjorn
bjorn previously approved these changes Feb 23, 2022
View reviewed changes
@aleokdev aleokdev requested a review from bjorn February 23, 2022 16:03
@aleokdev aleokdev merged commit f679def into mapeditor:master Feb 23, 2022
@aleokdev aleokdev added this to the 0.10.0 milestone Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bjorn bjorn bjorn approved these changes

Assignees
No one assigned
Labels
breaking This change breaks backwards-compatibility enhancement
Projects
None yet
Milestone
0.10.0
Development

Successfully merging this pull request may close these issues.
2 participants
@aleokdev @bjorn