-
Notifications
You must be signed in to change notification settings - Fork 21
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
tweak: Alumiumn Ore/Ingot #2
Comments
Thinking about this we should unify all ingots, ores, dusts and blends.
There are other items also, like plates, gears and more. |
Yeah, that is true and we could use the same across the board. I have re-done all the item textures in TE so we could use them as the master and copy over to the other mods? |
There is actually another (better?) way to do it in 1.12 rather than copying. Now we can change the models so they actually use the same physical textures. As an example, the Lithos pack does this beautifully. |
Yeah, I've seen other packs do it; but due to our patcher systems and people generally, download single patches rather than the full download. We'd need the files to be in all the individual mods folders. It's a little bit more effort but we can make a new repo which contains all the master files for ingots/ore etc... which then people can use for reference/copy paste. Unless we make a patch which is all the masters and then link to that using the model JSON's but that would require an update to the backend scripts etc... But could be an idea. |
Okay, I've re-written the backend script to handle this only took 3 days lol xD But I've started working on the master files and/or changes and will be pushing to this repo Soartex Ores and have started making the model files for the mods already in this repo. So I'll push all the changes at once. |
Another option would be using symbolic links |
The ingot and/or ore needs tweaking somewhat with some of the mods to help with the differences from Iron as it's close. Also, some people have issues seeing the changes. CC: @Darkosto
The text was updated successfully, but these errors were encountered: