Skip to content

Fixes Sprite IDs not assigned correctly on newer versions of Unity.#3

Open
CodeArtistMX wants to merge 1 commit into
prankard:masterfrom
CodeArtistMX:dev
Open

Fixes Sprite IDs not assigned correctly on newer versions of Unity.#3
CodeArtistMX wants to merge 1 commit into
prankard:masterfrom
CodeArtistMX:dev

Conversation

@CodeArtistMX
Copy link
Copy Markdown

@CodeArtistMX CodeArtistMX commented Apr 11, 2024

Updates the project to use the new UnityEditor.U2D.Sprites library when creating the sprite instances for the texture importer.

The previous SpriteMetaData class is now marked as obsolete and was not assigning IDs to sprites causing them to be unable to be selected individually in the Unity editor.

Here is more info about the issue:

https://forum.unity.com/threads/spritemetadata-textureimporter-potential-bug-in-unity-2021-2-16f1.1259632/

https://forum.unity.com/threads/example-code-for-ispriteeditordataprovider-as-a-replacement-for-textureimporter-spritesheets.1427991/

And thank you for creating this asset! 😄

…ity.

Updates the project to use the new UnityEditor.U2D.Sprites library when creating the sprite instances for the texture importer.
The previous SpriteMetaData class is now marked as obsolete and was not assigning IDs to sprites causing them to be unable to be selected individually in the Unity editor.
Here is more info about the issue:
https://forum.unity.com/threads/spritemetadata-textureimporter-potential-bug-in-unity-2021-2-16f1.1259632/

And thank you for creating this asset! 😄
@prankard
Copy link
Copy Markdown
Owner

Thanks for this.
I'll go through the code in a couple days to ensure it all still works with the few test files we have.
But it looks good from initial glance.
Thank you for submitting a fix!

@JonathanTremblay
Copy link
Copy Markdown
Contributor

I also made a new version. Mine is a bit different, but I think it solves the same problem! It also converts the tool to a package (no more files in the Assets folder).
My students have been using it for a few weeks, no complaints!
https://github.com/JonathanTremblay/FlashSpriteSheetImporter

@CodeArtistMX
Copy link
Copy Markdown
Author

Awesome!
The asset has been very helpful, so just wanted to bring this issue to your attention so others are able to keep using it in the future. Also as @JonathanTremblay mentioned, having it as a separate package in the future would be pretty cool too.

Thanks again for sharing your work, and if there is anything else I can do to help let me know!

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.

3 participants