-
Notifications
You must be signed in to change notification settings - Fork 4
Description
Currently both this repo's README, as well as https://github.com/CodeAndWeb/PhysicsEditor-CoronaSDK's README contain dead links to www.codeandweb.com as the site's URL structure has changed. However, I didn't address these links in my recent PRs as I would suggest that the tutorial names would also be changed (which would break the links again if they were updated now).
The website correctly lists Solar2D as a framework, https://www.codeandweb.com/texturepacker/tutorials/solar2d, but all of the Solar2D specific tutorials have "for CoronaSDK" in their titles. Their tags also list "corona", but not Solar2D. Inside the tutorials, there's CoronaSDK is also mentioned constantly, but Solar2D isn't.
Finally, as a part of my recent PR #1, I removed the long deprecated and unsupported SpriteGrabber module from the tutorial. This is module shouldn't need to be mentioned over at https://www.codeandweb.com/texturepacker/tutorials/example-with-coronasdk-using-physicseditor-texturepacker either to avoid confusion.
I understand that updating the website can be a pain, but the community (as well as the prospective newcomers) would greatly appreciate it.
Here's a quick list the of suggested changes for the website:
- update the repository names to read "Solar2D" instead of "CoronaSDK", see this current repository name.
- update the tutorial URL paths to read "solar2d" instead of "corona-sdk", case and point: https://www.codeandweb.com/texturepacker/tutorials/dynamic-content-scaling-for-corona-sdk
- update the tutorial titles to read Solar2D instead of CoronaSDK
- add solar2d tag to the tutorials
- update mentions to CoronaSDK inside the tutorials to say Solar2D instead
- remove mentions of SpriteGrabber in the "Example with CoronaSDK using PhysicsEditor & TexturePacker" tutorial
- finally, update the links to the tutorials in the GitHub repos