Skip to content

Added Point Object Support in Tiled. Replacement for PR(#6384) #7523

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 10 commits into from
Mar 9, 2025

Conversation

BoBIsHere86
Copy link
Contributor

Since there has been no activity from the original committer in over a year.
I pulled his PR, rebased and then added the necessary modifications to support it with the new json format as well as added the Point Object to the JsonObject test. (also one tiny parsing fix in the TMJLoader)

Other than that. This is the same PR as seen here #6384.
Once this is merged, we can close the old one.

@BoBIsHere86
Copy link
Contributor Author

Now that the libGDX jam is over. I am free to bug everyone about PRs again. Since #7534 was recently merged. This "point object" PR is the last of the current merge group.
Once this is merged I can move onto the final 3 tiled prs and update them against the latest master changes.

Let me know if there are any other issue's with this PR and I can fix it right up.

@BoBIsHere86 BoBIsHere86 force-pushed the feature-point-objects branch from ec5c585 to 94c4394 Compare January 6, 2025 17:34
@BoBIsHere86
Copy link
Contributor Author

There was a weird merge conflict with the latest CHANGES file. Updated it, Moved change to the top section with the other Breaking Change. All is well again.

@patton73
Copy link

Guys any news on this?

@BoBIsHere86
Copy link
Contributor Author

Guys any news on this?

Don't have any inside info, but on the bright side, it was added to the milestones for 1.13.2 a few weeks ago. So I think we can be assured it's not forgotten. We just have to sit back and wait for the merge, hoping sooner than later.

@Tom-Ski Tom-Ski merged commit a168e9e into libgdx:master Mar 9, 2025
2 checks passed
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.

7 participants