-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Conversation
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. Let me know if there are any other issue's with this PR and I can fix it right up. |
…h the BaseTmxMapLoader. Updated the Tiled Json Object test to have a point object to match the tmx test.
…and without odd indentation.
ec5c585
to
94c4394
Compare
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. |
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. |
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.